feat: CDrodownMenu: hide menu on esc

woothu · woothu · commit c4fea1c72777 · 2020-07-27T14:51:52.000+02:00
diff --git a/src/dropdown/CDropdownMenu.js b/src/dropdown/CDropdownMenu.js
@@ -63,11 +63,17 @@ const CDropdownMenu = props => {
     }
   }
 
+  const onKeypress = e => e.keyCode == '27' && setIsOpen(false)
+
   useEffect(() => {
     if (isOpen) {
       document.addEventListener('click', checkClose)
+      document.addEventListener('keydown', onKeypress)
+    }
+    return () => {
+      document.removeEventListener('click', checkClose)
+      document.removeEventListener('keydown', onKeypress) 
     }
-    return () => document.removeEventListener('click', checkClose)
   }, [isOpen])
 
   return (

Original file line number	Diff line number	Diff line change
`@@ -63,11 +63,17 @@ const CDropdownMenu = props => {`
`63`	`63`	`}`
`64`	`64`	`}`
`65`	`65`
	`66`	`+ const onKeypress = e => e.keyCode == '27' && setIsOpen(false)`
	`67`	`+`
`66`	`68`	`useEffect(() => {`
`67`	`69`	`if (isOpen) {`
`68`	`70`	`document.addEventListener('click', checkClose)`
	`71`	`+ document.addEventListener('keydown', onKeypress)`
	`72`	`+ }`
	`73`	`+ return () => {`
	`74`	`+ document.removeEventListener('click', checkClose)`
	`75`	`+ document.removeEventListener('keydown', onKeypress)`
`69`	`76`	`}`
`70`		`- return () => document.removeEventListener('click', checkClose)`
`71`	`77`	`}, [isOpen])`
`72`	`78`
`73`	`79`	`return (`