added requested changes

Pujan92 · Jan 10, 2024 · 1b22ed2 · 1b22ed2
1 parent 325073d
commit 1b22ed2
Showing 1 changed file with 8 additions and 9 deletions.
diff --git a/src/pages/KeyboardShortcutsPage.tsx b/src/pages/KeyboardShortcutsPage.tsx
@@ -8,25 +8,24 @@ import useLocalize from '@hooks/useLocalize';
 import useThemeStyles from '@hooks/useThemeStyles';
 import KeyboardShortcut from '@libs/KeyboardShortcut';
 import CONST from '@src/CONST';
-import type {TranslationPaths} from '@src/languages/types';
+
+type Shortcut = {
+    displayName: string;
+    descriptionKey: 'search' | 'newChat' | 'openShortcutDialog' | 'escape' | 'copy';
+};
 
 function KeyboardShortcutsPage() {
     const styles = useThemeStyles();
     const {translate} = useLocalize();
     const shortcuts = Object.values(CONST.KEYBOARD_SHORTCUTS)
-        .filter((shortcut) => shortcut.descriptionKey !== undefined && shortcut.descriptionKey !== null)
         .map((shortcut) => {
             const platformAdjustedModifiers = KeyboardShortcut.getPlatformEquivalentForKeys(shortcut.modifiers);
             return {
                 displayName: KeyboardShortcut.getDisplayName(shortcut.shortcutKey, platformAdjustedModifiers),
                 descriptionKey: shortcut.descriptionKey,
             };
-        });
-
-    type Shortcut = {
-        displayName: string;
-        descriptionKey: string | null;
-    };
+        })
+        .filter((shortcut): shortcut is Shortcut => !!shortcut.descriptionKey);
     /**
      * Render the information of a single shortcut
      * @param shortcut - The shortcut to render
@@ -35,7 +34,7 @@ function KeyboardShortcutsPage() {
         <MenuItem
             key={shortcut.displayName}
             title={shortcut.displayName}
-            description={translate(`keyboardShortcutsPage.shortcuts.${shortcut.descriptionKey}` as TranslationPaths)}
+            description={translate(`keyboardShortcutsPage.shortcuts.${shortcut.descriptionKey}`)}
             wrapperStyle={[styles.ph0, styles.cursorAuto]}
             interactive={false}
         />