Don't init until readyState is complete, to prevent missing document.…

…body errors in IE. H/T @matthargett.
HubSpot · Dec 9, 2013 · 55c8c84 · 55c8c84
1 parent c123ab2
commit 55c8c84
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/coffee/ui.coffee b/coffee/ui.coffee
@@ -121,7 +121,7 @@ init = ->
     flashClass 'offline-ui-reconnect-succeeded-2s', 2
     flashClass 'offline-ui-reconnect-succeeded-5s', 5
 
-if document.readyState in ['interactive', 'complete']
+if document.readyState is 'complete'
   init()
 else if document.addEventListener?
   document.addEventListener 'DOMContentLoaded', init, false

diff --git a/js/ui.js b/js/ui.js
@@ -1,5 +1,5 @@
 (function() {
-  var RETRY_TEMPLATE, TEMPLATE, addClass, content, createFromHTML, el, flashClass, flashTimeouts, formatTime, init, removeClass, render, _onreadystatechange, _ref;
+  var RETRY_TEMPLATE, TEMPLATE, addClass, content, createFromHTML, el, flashClass, flashTimeouts, formatTime, init, removeClass, render, _onreadystatechange;
 
   if (!window.Offline) {
     throw new Error("Offline UI brought in without offline.js");
@@ -138,7 +138,7 @@
     });
   };
 
-  if ((_ref = document.readyState) === 'interactive' || _ref === 'complete') {
+  if (document.readyState === 'complete') {
     init();
   } else if (document.addEventListener != null) {
     document.addEventListener('DOMContentLoaded', init, false);

diff --git a/offline.min.js b/offline.min.js