Skip to content
Permalink
Browse files

From a suggestion by Diego, moved the jQuery.ready() call outside of …

…the try{}catch() block for IE/Safari DOM Ready.
  • Loading branch information
jeresig committed Oct 15, 2007
1 parent 971840f commit a2fc65fd3296931486974a3b362949ebc1a01062
Showing with 12 additions and 16 deletions.
  1. +12 −16 src/event.js
@@ -430,24 +430,20 @@ function bindReady(){
document.addEventListener( "DOMContentLoaded", jQuery.ready, false );

// If Safari or IE is used
else {
// Continually check to see if the document is ready
function timer(){
try {
// If IE is used, use the trick by Diego Perini
// http://javascript.nwbox.com/IEContentLoaded/
if ( jQuery.browser.msie || document.readyState != "loaded" && document.readyState != "complete" )
document.documentElement.doScroll("left");

// and execute any waiting functions
jQuery.ready();
} catch( error ) {
setTimeout( timer, 0 );
}
// Continually check to see if the document is ready
else (function(){
try {
// If IE is used, use the trick by Diego Perini
// http://javascript.nwbox.com/IEContentLoaded/
if ( jQuery.browser.msie || document.readyState != "loaded" && document.readyState != "complete" )
document.documentElement.doScroll("left");
} catch( error ) {
return setTimeout( arguments.callee, 0 );
}

timer();
}
// and execute any waiting functions
jQuery.ready();
})();

// A fallback to window.onload, that will always work
jQuery.event.add( window, "load", jQuery.ready );

0 comments on commit a2fc65f

Please sign in to comment.
You can’t perform that action at this time.