fix(popper): account for touchstart

TheSharpieOne · TheSharpieOne · commit 9b80d1144732 · 2017-09-16T09:20:41.000-04:00
fixes dropdowns, tooltips, and popovers not triggers on touch events Closes #456 #458
diff --git a/src/Dropdown.js b/src/Dropdown.js
@@ -66,11 +66,15 @@ class Dropdown extends React.Component {
   }
 
   addEvents() {
-    document.addEventListener('click', this.handleDocumentClick, true);
+    ['click', 'touchstart'].forEach(event =>
+      document.addEventListener(event, this.handleDocumentClick, true)
+    );
   }
 
   removeEvents() {
-    document.removeEventListener('click', this.handleDocumentClick, true);
+    ['click', 'touchstart'].forEach(event =>
+      document.removeEventListener(event, this.handleDocumentClick, true)
+    );
   }
 
   handleDocumentClick(e) {
diff --git a/src/Popover.js b/src/Popover.js
@@ -121,11 +121,15 @@ class Popover extends React.Component {
   }
 
   addTargetEvents() {
-    document.addEventListener('click', this.handleDocumentClick, true);
+    ['click', 'touchstart'].forEach(event =>
+      document.addEventListener(event, this.handleDocumentClick, true)
+    );
   }
 
   removeTargetEvents() {
-    document.removeEventListener('click', this.handleDocumentClick, true);
+    ['click', 'touchstart'].forEach(event =>
+      document.removeEventListener(event, this.handleDocumentClick, true)
+    );
   }
 
   toggle(e) {
diff --git a/src/Tooltip.js b/src/Tooltip.js
@@ -144,13 +144,17 @@ class Tooltip extends React.Component {
   addTargetEvents() {
     this._target.addEventListener('mouseover', this.onMouseOverTooltip, true);
     this._target.addEventListener('mouseout', this.onMouseLeaveTooltip, true);
-    document.addEventListener('click', this.handleDocumentClick, true);
+    ['click', 'touchstart'].forEach(event =>
+      document.addEventListener(event, this.handleDocumentClick, true)
+    );
   }
 
   removeTargetEvents() {
     this._target.removeEventListener('mouseover', this.onMouseOverTooltip, true);
     this._target.removeEventListener('mouseout', this.onMouseLeaveTooltip, true);
-    document.removeEventListener('click', this.handleDocumentClick, true);
+    ['click', 'touchstart'].forEach(event =>
+      document.removeEventListener(event, this.handleDocumentClick, true)
+    );
   }
 
   toggle(e) {

Original file line number	Diff line number	Diff line change
`@@ -66,11 +66,15 @@ class Dropdown extends React.Component {`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`addEvents() {`
`69`		`- document.addEventListener('click', this.handleDocumentClick, true);`
	`69`	`+ ['click', 'touchstart'].forEach(event =>`
	`70`	`+ document.addEventListener(event, this.handleDocumentClick, true)`
	`71`	`+ );`
`70`	`72`	`}`
`71`	`73`
`72`	`74`	`removeEvents() {`
`73`		`- document.removeEventListener('click', this.handleDocumentClick, true);`
	`75`	`+ ['click', 'touchstart'].forEach(event =>`
	`76`	`+ document.removeEventListener(event, this.handleDocumentClick, true)`
	`77`	`+ );`
`74`	`78`	`}`
`75`	`79`
`76`	`80`	`handleDocumentClick(e) {`
Original file line number	Diff line number	Diff line change
`@@ -121,11 +121,15 @@ class Popover extends React.Component {`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`addTargetEvents() {`
`124`		`- document.addEventListener('click', this.handleDocumentClick, true);`
	`124`	`+ ['click', 'touchstart'].forEach(event =>`
	`125`	`+ document.addEventListener(event, this.handleDocumentClick, true)`
	`126`	`+ );`
`125`	`127`	`}`
`126`	`128`
`127`	`129`	`removeTargetEvents() {`
`128`		`- document.removeEventListener('click', this.handleDocumentClick, true);`
	`130`	`+ ['click', 'touchstart'].forEach(event =>`
	`131`	`+ document.removeEventListener(event, this.handleDocumentClick, true)`
	`132`	`+ );`
`129`	`133`	`}`
`130`	`134`
`131`	`135`	`toggle(e) {`