Merge pull request #19 from amccloud/patch-2

Check for addEventListener before trying to use to support IE 8
reactjs · Mar 24, 2015 · bb57045 · bb57045
2 parents 10e9582 + c464368
commit bb57045
Showing 1 changed file with 17 additions and 4 deletions.
diff --git a/lib/helpers/focusManager.js b/lib/helpers/focusManager.js
@@ -40,13 +40,26 @@ exports.returnFocus = function() {
 
 exports.setupScopedFocus = function(element) {
   modalElement = element;
-  window.addEventListener('blur', handleBlur, false);
-  document.addEventListener('focus', handleFocus, true);
+
+  if (window.addEventListener) {
+    window.addEventListener('blur', handleBlur, false);
+    document.addEventListener('focus', handleFocus, true);
+  } else {
+    window.attachEvent('onBlur', handleBlur);
+    document.attachEvent('onFocus', handleFocus);
+  }
 };
 
 exports.teardownScopedFocus = function() {
   modalElement = null;
-  window.removeEventListener('blur', handleBlur);
-  document.removeEventListener('focus', handleFocus);
+
+  if (window.addEventListener) {
+    window.removeEventListener('blur', handleBlur);
+    document.removeEventListener('focus', handleFocus);
+  } else {
+    window.detachEvent('onBlur', handleBlur);
+    document.detachEvent('onFocus', handleFocus);
+  }
 };
 
+