Ticket #2531547 (accepted defect)

Reporter


Standa Opichal
Opened: 12/6/11
Last modified: 07/13/12
Status: accepted
Type: defect

Owner


Luke Smith
Target Release:
Priority: P3 (normal)
Summary: event-focus crashes in IE7/IE8
Description:

It received 'beforeactivate' DOM event with toElement === null which causes the code to fail.

Type: defect Observed in Version: 3.4.1
Component: Event Severity: S3 (normal)
Assigned To: Luke Smith Target Release:
Location: Priority: P3 (normal)
Tags: Relates To:
Browsers: IE 7.x,IE 8.x
URL: https://github.com/opichals/yui3/tree/ticket_2531547_event_focus_ie78_null_e_target
Test Information:

Change History

Standa Opichal

Posted: 12/6/11
  • testurl changed to https://github.com/opichals/yui3/tree/ticket_2531547_event_focus_ie78_null_e_target

Standa Opichal

Posted: 12/6/11
  • component changed from None to Event
  • owner changed from Thomas S. Sha to Luke Smith
  • status changed from new to assigned

Luke Smith

YUI Contributor

Posted: 12/6/11
  • priority changed to P3 (normal)
  • status changed from assigned to accepted

Were you able to create a reduced reproduction of the issue? The _proxy handler should be receiving an event facade (e) that has passed through the DOMEventFacade (IE version) constructor, which *really should* populate e.target. I'm curious about the root of the problem.

focuzz

Posted: 07/2/12

I got same problem, you can debug it in ie10 (windows 8 rc) in ie7 compatibility mode or in ie7 (on windows xp). e.target is undefined, so target.getData(nodeDataKey) fails. I got this error in mojito, using binder - ie7(ie8 with all service packs not) fails.

focuzz

Posted: 07/2/12

I got same problem, you can debug it in ie10 (windows 8 rc) in ie7 compatibility mode or in ie7 (on windows xp). e.target is undefined, so target.getData(nodeDataKey) fails. I got this error in mojito, using binder - ie7(ie8 with all service packs not) fails.

Luke Smith

YUI Contributor

Posted: 07/12/12

Can you update the test url to an endpoint? That link points to the root of a branch.

focuzz

Posted: 07/13/12

Just got this error. It takes much time to reproduce it. I've experimented that time, so it takes some time to remember.
I've updated https://github.com/focuzz/mj-defer-demo , but it's not enough to reproduce.
After you pull and start this demo (root url - '/'), you'll get in IE7 error I've described there https://github.com/yahoo/mojito/issues/262
Then, I've tried to fix this error in this way:
in mojito/lib/app/autoload/output-handler.client.js on line 101,
I replaced
if (meta.http.headers['content-type'] && meta.http.headers['content-type'][0].indexOf('application/json') === 0)
with
if (meta.http.headers['content-type'] && Y.Lang.isArray(meta.http.headers['content-type']) && meta.http.headers['content-type'][0].indexOf('application/json') === 0)
This helps with initial error in IE7, but, if you click on any link in menu in IE7 after this - you'll get error described above.

If you'll have any questions - feel free to ask.