fix: Adding a cleanup function to add a removeEventListener to preven…

…t hangs while pressing ESC
carbon-design-system · Sep 10, 2020 · fbc9f0c · fbc9f0c
1 parent e9d2f53
commit fbc9f0c
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/packages/gatsby-theme-carbon/src/components/GlobalSearch/GlobalSearch.js b/packages/gatsby-theme-carbon/src/components/GlobalSearch/GlobalSearch.js
@@ -79,12 +79,19 @@ const GlobalSearchInput = () => {
   );
 
   useEffect(() => {
-    document.addEventListener('keyup', function(e){ 
+    var collapseOpenNavs = function(e){ 
       if(e.which == 27){
         toggleNavState('switcherIsOpen', 'close');
         toggleNavState('searchIsOpen', 'close');
       }  
-    })
+    };
+
+    document.addEventListener('keyup',collapseOpenNavs)
+
+    return function cleanup(){
+      document.removeEventListener('keyup', collapseOpenNavs)
+    };
+
   })
 
   useEffect(() => {