Merge pull request react-bootstrap#117 from react-bootstrap/rcw-doc

Fix rootClose on unmounting components

Author: taion

src/RootCloseWrapper.js

@@ -44,8 +44,11 @@ export default class RootCloseWrapper extends React.Component {
   bindRootCloseHandlers() {
     const doc = ownerDocument(this);
 
+    // Use capture for this listener so it fires before React's listener, to
+    // avoid false positives in the contains() check below if the target DOM
+    // element is removed in the React mouse callback.
     this._onDocumentMouseListener =
-      addEventListener(doc, this.props.event, this.handleDocumentMouse);
+      addEventListener(doc, this.props.event, this.handleDocumentMouse, true);
 
     this._onDocumentKeyupListener =
       addEventListener(doc, 'keyup', this.handleDocumentKeyUp);

src/utils/addEventListener.js

@@ -1,11 +1,11 @@
 import addEventListener from 'dom-helpers/events/on';
 import removeEventListener from 'dom-helpers/events/off';
 
-export default function (node, event, handler) {
-  addEventListener(node, event, handler);
+export default function (node, event, handler, capture) {
+  addEventListener(node, event, handler, capture);
   return {
     remove(){
-      removeEventListener(node, event, handler);
+      removeEventListener(node, event, handler, capture);
     }
   };
 }