Search and Top Navigation
#11539 closed bug (fixed)
Opened March 18, 2015 05:01PM UTC
Closed March 24, 2015 10:28PM UTC
Last modified March 25, 2015 05:21PM UTC
Core: safeBlur breaks in IE11
Reported by: | jzaefferer | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | git |
Component: | ui.core | Version: | git (not yet released) |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
This is a regression from a recent change ( https://github.com/jquery/jquery-ui/commit/f33027840cdac5152599da66635981bbe68c6bda ), causing three test suites to fail in IE11. From a recent build:
- Dialog: http://swarm.jquery.org/result/81652
- Draggable: http://swarm.jquery.org/result/80673
- Droppable: http://swarm.jquery.org/result/80668
The error is the same for each:
Unable to get property 'toLowerCase' of undefined or null reference
Source: https://github.com/jquery/jquery-ui/blob/8bcd363ae640bbefc28b720c1ab58b7992ea7044/ui/core.js#L74
Only dialog and draggable use safeBlur. Can't yet tell why
elementis defined, but
element.nodeNameisn't.
Attachments (0)
Change History (10)
Changed March 18, 2015 06:28PM UTC by comment:1
Changed March 18, 2015 06:32PM UTC by comment:2
Probably related to the iframe TestSwarm uses. There the error is showing up pretty consistently over multiple builds.
Changed March 24, 2015 10:28PM UTC by comment:3
resolution: | → fixed |
---|---|
status: | new → closed |
Core: Handle IE11 bug with document.activeElement in iframes
Fixes #11539
Closes gh-1518
Changeset: 76c27556f48bea48d3787c241d35e190d46c3245
Changed March 24, 2015 11:46PM UTC by comment:4
milestone: | none → git |
---|
Changed March 25, 2015 03:08PM UTC by comment:5
I spent a while trying to reduce this as much as I could, but I gave up after a while. Here's what I came up with: http://jsbin.com/fejituname/1/. It still has a dependency on jQuery Core, but it reliably reproduces the bug.
Changed March 25, 2015 04:44PM UTC by comment:6
What should I see from that test case?
Changed March 25, 2015 04:51PM UTC by comment:7
You should see an alert with an error message in IE11.
Changed March 25, 2015 04:59PM UTC by comment:8
I don't see an alert in either real Win8.1+IE11 on Windows or on Win7+IE11 Browserstack.
Did I mention I hate focus?
Changed March 25, 2015 05:16PM UTC by comment:9
That's because of the new live preview time limit in jsbin :-/ It loads the editor instead of the actual page now.
Changed March 25, 2015 05:21PM UTC by comment:10
http://jsbin.com/towote/1 is a new clone of the bin with a registered account, so it should work forever.
I can't reproduce the failures even when using the TestSwarm URLs.