Add code comment clarifying where parallel change to shortcut documen…

…tation should be
kommunarr · Dec 17, 2024 · 0126970 · 0126970
1 parent 241dfc0
commit 0126970
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/constants.js b/src/constants.js
@@ -117,7 +117,11 @@ const SyncEvents = {
   },
 }
 
-// note: the multi-key shortcut values are currently just for display use in action titles
+/*
+  DEV NOTE: Duplicate any and all changes made here to our [official documentation site here](https://github.com/FreeTubeApp/FreeTube-Docs/blob/master/usage/keyboard-shortcuts.md)
+  to have them reflect on the [keyboard shortcut reference webpage](https://docs.freetubeapp.io/usage/keyboard-shortcuts).
+  Please also update the [keyboard shortcut modal](src/renderer/components/FtKeyboardShortcutPrompt/FtKeyboardShortcutPrompt.vue)
+*/
 const KeyboardShortcuts = {
   APP: {
     GENERAL: {

diff --git a/src/renderer/helpers/utils.js b/src/renderer/helpers/utils.js
@@ -969,7 +969,7 @@ export function getLocalizedShortcut(shortcut) {
   const shortcuts = shortcut.split('+')
 
   if (process.platform === 'darwin') {
-    const shortcutsAsIcons = shortcuts.map(shortCut => getMacIconForShortcut(shortCut))
+    const shortcutsAsIcons = shortcuts.map(shortcut => getMacIconForShortcut(shortcut))
     return shortcutsAsIcons.join('')
   } else {
     const localizedShortcuts = shortcuts.map((shortcut) => getIndividualLocalizedShortcut(shortcut))