fix: fixes onClickOutside on iOS devices. fixes #13

theKashey · Aug 14, 2019 · 96d0756 · 96d0756
1 parent e886d62
commit 96d0756
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/Effect.tsx b/src/Effect.tsx
@@ -37,7 +37,7 @@ export function Effect(
       }
     };
 
-    const onClick = (event: MouseEvent) => {
+    const onClick = (event: MouseEvent | TouchEvent) => {
       if (event.defaultPrevented || event.target === lastEventTarget) {
         return;
       }
@@ -65,6 +65,7 @@ export function Effect(
       }
       document.addEventListener('keyup', onKeyPress);
       document.addEventListener('click', onClick);
+      document.addEventListener('touchstart', onClick);
     };
 
     const onNodeDeactivation = () => {
@@ -74,12 +75,16 @@ export function Effect(
       }
       document.removeEventListener('keyup', onKeyPress);
       document.removeEventListener('click', onClick);
+      document.removeEventListener('touchstart', onClick);
     };
 
     setLockProps({
       onClick: (e: React.MouseEvent) => {
         lastEventTarget = e.target
       },
+      onTouchStart: (e: React.TouchEvent) => {
+        lastEventTarget = e.target
+      },
       onActivation: onNodeActivation,
       onDeactivation: onNodeDeactivation,
     });

diff --git a/src/types.ts b/src/types.ts
@@ -3,6 +3,8 @@ import * as React from "react";
 export interface LockProps {
   onClick?(e: React.MouseEvent): void;
 
+  onTouchStart?(e: React.TouchEvent): void;
+
   onActivation?(node: HTMLElement): void;
 
   onDeactivation?(): void;
@@ -38,4 +40,4 @@ export interface ReactFocusOnSideProps extends ReactFocusOnProps {
 
 export interface EffectProps extends CommonProps {
   setLockProps(settings: LockProps): void;
-}
+}