jed / already.js Goto Github PK

Both @ded and you are looking for a value that contains "in", however the ready states are uninitialized, loading, loaded, *interactive*, and complete. This can cause a match for interactive and ignore it as "ready".

So, something like this avoids assuming readyState is a string that has an indexOf method:

 /ded|ve|ete/.test(document.readyState);

Note:

I believe the practice of checking for a loaded value was started by @jeresig for older versions of Safari that would return loaded for cached pages (thinking Safari 2, though I failed to reproduce on my old iMac G4).

However, I'm skeptical of allowing loaded to slip through as safe for all browsers (thinking of IE) as the doScroll-technique suggests the earliest usable state for IE is interactive, not loaded, as seen by this test.

The event is DOMContentLoaded not domContentLoaded.

The /m/( document.readystate ) hottness doesn't actually work in IE, at least in my testing.