fix: set capture on event listeners

dequelabs · Sep 30, 2024 · 469811e · 469811e
1 parent 56107e7
commit 469811e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/packages/react/src/components/ClickOutsideListener/index.tsx b/packages/react/src/components/ClickOutsideListener/index.tsx
@@ -60,15 +60,15 @@ function ClickOutsideListener(
 
   useEffect(() => {
     typeof mouseEvent === 'string' &&
-      document.addEventListener(mouseEvent, handleEvent);
+      document.addEventListener(mouseEvent, handleEvent, true);
     typeof touchEvent === 'string' &&
-      document.addEventListener(touchEvent, handleEvent);
+      document.addEventListener(touchEvent, handleEvent, true);
 
     return () => {
       typeof mouseEvent === 'string' &&
-        document.removeEventListener(mouseEvent, handleEvent);
+        document.removeEventListener(mouseEvent, handleEvent, true);
       typeof touchEvent === 'string' &&
-        document.removeEventListener(touchEvent, handleEvent);
+        document.removeEventListener(touchEvent, handleEvent, true);
     };
   }, [mouseEvent, touchEvent]);
 

diff --git a/packages/react/src/components/Popover/index.tsx b/packages/react/src/components/Popover/index.tsx
@@ -189,8 +189,6 @@ const Popover = forwardRef<HTMLDivElement, PopoverProps>(
     }, [targetElement, id, show]);
 
     const handleClickOutside = () => {
-      // console.log("should the initial click be calling handleClickOutside?");
-      // console.log('handleClickOutside', show);
       if (show) {
         handleClosePopover();
       }