From dabd5ba96c05279b3ffb052db5b8d17f75996694 Mon Sep 17 00:00:00 2001 From: Timmy Willison Date: Wed, 28 Oct 2015 13:54:04 -0400 Subject: [PATCH] Core: use interactive to evaluate dom ready, barring IE9-10 Fixes gh-2100 --- src/core/ready.js | 8 ++++---- test/data/event/interactiveReady.html | 23 +++++++++++++++++++++++ test/unit/event.js | 12 ++++++++++++ 3 files changed, 39 insertions(+), 4 deletions(-) create mode 100644 test/data/event/interactiveReady.html diff --git a/src/core/ready.js b/src/core/ready.js index 7a7ef5869..73c3d706a 100644 --- a/src/core/ready.js +++ b/src/core/ready.js @@ -70,10 +70,10 @@ jQuery.ready.promise = function( obj ) { // Catch cases where $(document).ready() is called // after the browser event has already occurred. - // We once tried to use readyState "interactive" here, - // but it caused issues like the one - // discovered by ChrisS here: http://bugs.jquery.com/ticket/12282#comment:15 - if ( document.readyState === "complete" ) { + // Support: IE9-10 only + // Older IE sometimes signals "interactive" too soon + if ( document.readyState === "complete" || + ( document.readyState !== "loading" && !document.documentElement.doScroll ) ) { // Handle it asynchronously to allow scripts the opportunity to delay ready window.setTimeout( jQuery.ready ); diff --git a/test/data/event/interactiveReady.html b/test/data/event/interactiveReady.html new file mode 100644 index 000000000..77b37104d --- /dev/null +++ b/test/data/event/interactiveReady.html @@ -0,0 +1,23 @@ + + + + +Test case for gh-2100 + + + + + + + + +
+ + diff --git a/test/unit/event.js b/test/unit/event.js index ebaf42f6b..479f805db 100644 --- a/test/unit/event.js +++ b/test/unit/event.js @@ -2518,6 +2518,18 @@ testIframeWithCallback( } ); +// need PHP here to make the incepted IFRAME hang +if ( hasPHP ) { + testIframeWithCallback( + "jQuery.ready uses interactive", + "event/interactiveReady.html", + function( isOk, assert ) { + assert.expect( 1 ); + assert.ok( isOk, "jQuery fires ready when the DOM can truly be interacted with" ); + } + ); +} + testIframeWithCallback( "Focusing iframe element", "event/focusElem.html",