Skip to main content

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:

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

element
is defined, but
element.nodeName
isn't.

Attachments (0)
Change History (10)

Changed March 18, 2015 06:28PM UTC by scottgonzalez comment:1

I can't reproduce the failures even when using the TestSwarm URLs.

Changed March 18, 2015 06:32PM UTC by jzaefferer 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 Scott González comment:3

resolution: → fixed
status: newclosed

Core: Handle IE11 bug with document.activeElement in iframes

Fixes #11539

Closes gh-1518

Changeset: 76c27556f48bea48d3787c241d35e190d46c3245

Changed March 24, 2015 11:46PM UTC by scottgonzalez comment:4

milestone: nonegit

Changed March 25, 2015 03:08PM UTC by scottgonzalez 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 dmethvin comment:6

What should I see from that test case?

Changed March 25, 2015 04:51PM UTC by scottgonzalez comment:7

You should see an alert with an error message in IE11.

Changed March 25, 2015 04:59PM UTC by dmethvin 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 scottgonzalez 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 scottgonzalez comment:10

http://jsbin.com/towote/1 is a new clone of the bin with a registered account, so it should work forever.