fix(react): use capture phase instead of bubble phase for esc keypres…

…s on Tooltip (#1569)
dequelabs · Jul 10, 2024 · 1bc0e02 · 1bc0e02
1 parent 66bea50
commit 1bc0e02
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/packages/react/src/components/Tooltip/index.tsx b/packages/react/src/components/Tooltip/index.tsx
@@ -136,13 +136,17 @@ export default function Tooltip({
 
     const targetElement = document.body;
     if (showTooltip) {
-      targetElement.addEventListener('keyup', handleEscape);
+      targetElement.addEventListener('keyup', handleEscape, { capture: true });
     } else {
-      targetElement.removeEventListener('keyup', handleEscape);
+      targetElement.removeEventListener('keyup', handleEscape, {
+        capture: true
+      });
     }
 
     return () => {
-      targetElement.removeEventListener('keyup', handleEscape);
+      targetElement.removeEventListener('keyup', handleEscape, {
+        capture: true
+      });
     };
   }, [showTooltip]);