Merge pull request #1187 from OskarDamkjaer/esc_shortcut

Re-add esc as shortcut
neo4j · Sep 7, 2020 · f301b5f · f301b5f
2 parents f4fde21 + 16c9cf3
commit f301b5f
Showing 1 changed file with 14 additions and 3 deletions.
diff --git a/src/browser/modules/App/keyboardShortcuts.ts b/src/browser/modules/App/keyboardShortcuts.ts
@@ -17,6 +17,10 @@ interface Shortcut {
   modifyers: ModifierKey[]
   key: string
 }
+export const OLD_FULLSCREEN_SHORTCUT: Shortcut = {
+  modifyers: [],
+  key: 'Escape'
+}
 export const FULLSCREEN_SHORTCUT: Shortcut = {
   modifyers: [modKey, 'altKey'],
   key: 'f'
@@ -65,8 +69,11 @@ export function useKeyboardShortcuts(bus: Bus): void {
     }
   }
 
-  const expandEditor = (e: KeyboardEvent): void => {
-    if (matchesShortcut(e, FULLSCREEN_SHORTCUT)) {
+  const fullscreenEditor = (e: KeyboardEvent): void => {
+    if (
+      matchesShortcut(e, FULLSCREEN_SHORTCUT) ||
+      matchesShortcut(e, OLD_FULLSCREEN_SHORTCUT)
+    ) {
       e.preventDefault()
       trigger(EXPAND)
     }
@@ -79,7 +86,11 @@ export function useKeyboardShortcuts(bus: Bus): void {
     }
   }
 
-  const keyboardShortcuts = [focusEditorOnSlash, expandEditor, cardSizeShortcut]
+  const keyboardShortcuts = [
+    focusEditorOnSlash,
+    fullscreenEditor,
+    cardSizeShortcut
+  ]
 
   useEffect(() => {
     keyboardShortcuts.forEach(shortcut =>