fix(modal): fix event listener for modal (#127)

TheSharpieOne · eddywashere · commit c778608f7153 · 2016-09-07T21:25:40.000-07:00
Fixes #126 Change the event listeners for click and keyup in modal to set `useCapture` to true to trigger the handler before any other handler.
diff --git a/src/Modal.js b/src/Modal.js
@@ -100,7 +100,7 @@ class Modal extends React.Component {
   }
 
   removeEvents() {
-    document.removeEventListener('click', this.handleBackdropClick, false);
+    document.removeEventListener('click', this.handleBackdropClick, true);
     document.removeEventListener('keyup', this.handleEscape, false);
   }
 
@@ -115,7 +115,7 @@ class Modal extends React.Component {
     this._element.setAttribute('tabindex', '-1');
 
     document.body.appendChild(this._element);
-    document.addEventListener('click', this.handleBackdropClick, false);
+    document.addEventListener('click', this.handleBackdropClick, true);
     document.addEventListener('keyup', this.handleEscape, false);
 
     document.body.className = classNames(

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ class Modal extends React.Component {`
`100`	`100`	`}`
`101`	`101`
`102`	`102`	`removeEvents() {`
`103`		`- document.removeEventListener('click', this.handleBackdropClick, false);`
	`103`	`+ document.removeEventListener('click', this.handleBackdropClick, true);`
`104`	`104`	`document.removeEventListener('keyup', this.handleEscape, false);`
`105`	`105`	`}`
`106`	`106`
`@@ -115,7 +115,7 @@ class Modal extends React.Component {`
`115`	`115`	`this._element.setAttribute('tabindex', '-1');`
`116`	`116`
`117`	`117`	`document.body.appendChild(this._element);`
`118`		`- document.addEventListener('click', this.handleBackdropClick, false);`
	`118`	`+ document.addEventListener('click', this.handleBackdropClick, true);`
`119`	`119`	`document.addEventListener('keyup', this.handleEscape, false);`
`120`	`120`
`121`	`121`	`document.body.className = classNames(`