#16437 closed defect (fixed)
exception in IE relating to focus event when browser window reselected
Reported by: | DaveHS | Owned by: | bill |
---|---|---|---|
Priority: | undecided | Milestone: | 1.8.6 |
Component: | Dijit | Version: | 1.8.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
We have a grid within a tab container - double clicking on items in the grid opens a details record in a new tab. If the user clicks away from the browser at this point e.g. switches to another open application, then switches back to the browser, a javascript error is reported:
SCRIPT5007: Unable to get value of the property 'toLowerCase': object is null or undefined
This only happens if the browser is in strict mode (example attached).
We have traced this to the dijit focus.js code - a one line change is all that is required to fix it (we can supply details).
Attachments (2)
Change History (10)
Changed 8 years ago by
Attachment: | index.html added |
---|
Changed 8 years ago by
Attachment: | focus.js.uncompressed.js.patch added |
---|
Patch file with single line change to prevent the error.
comment:1 follow-up: 2 Changed 8 years ago by
Milestone: | tbd → 1.9 |
---|---|
Status: | new → assigned |
OK, I'll take a look. Thanks for the test file and patch.\
Do you happen to know what strange element is getting the focusin event? There's no tagName so I wonder if it's a text node or ... ?
comment:2 Changed 8 years ago by
Replying to bill:
OK, I'll take a look. Thanks for the test file and patch.\
Do you happen to know what strange element is getting the focusin event? There's no tagName so I wonder if it's a text node or ... ?
Sorry, no idea - we can see that the event's srcElement object doesn't have a tagName - it looks like an empty object in the debugger... The last item to (sucessfully) have the focus is the TD from the grid.
Thanks for looking at it!
comment:3 Changed 8 years ago by
Summary: | Javascript console error in IE (9) relating to focus event → exception in IE relating to focus event when browser window reselected |
---|
Indeed, very strange. OK, sure, I can check that in. BTW this happens in IE10 too.
The error is not critical since it aborts the exception aborts the handler in the same as the "return" statement in your patch, but I'll glady check it in anyway, to keep the console clean.
comment:8 Changed 8 years ago by
Milestone: | 1.9 → 1.8.6 |
---|
Sample file showing the issue