Merge pull request react-bootstrap#755 from bbc/dropdown-close-ie8-fix

[changed] Fix button dropdown not closing when clicking button on IE8
tomlelouch · Jun 1, 2015 · fc378de · fc378de
2 parents 2473fae + 0cfbf3b
commit fc378de
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/DropdownStateMixin.js b/src/DropdownStateMixin.js
@@ -49,7 +49,9 @@ const DropdownStateMixin = {
   handleDocumentClick(e) {
     // If the click originated from within this component
     // don't do anything.
-    if (isNodeInRoot(e.target, React.findDOMNode(this))) {
+    // e.srcElement is required for IE8 as e.target is undefined
+    let target = e.target || e.srcElement;
+    if (isNodeInRoot(target, React.findDOMNode(this))) {
       return;
     }
 

diff --git a/src/RootCloseWrapper.js b/src/RootCloseWrapper.js
@@ -42,7 +42,9 @@ export default class RootCloseWrapper extends React.Component {
 
   handleDocumentClick(e) {
     // If the click originated from within this component, don't do anything.
-    if (isNodeInRoot(e.target, React.findDOMNode(this))) {
+    // e.srcElement is required for IE8 as e.target is undefined
+    let target = e.target || e.srcElement;
+    if (isNodeInRoot(target, React.findDOMNode(this))) {
       return;
     }