ByoungSooPark/modeling-app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Change shortcut separator to be + and no spaces

Browse Source
This commit is contained in:
Jonathan Tran
2024-08-22 13:59:35 -04:00
parent af7cb69831
commit 61f6e7bc99
2 changed files with 14 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
e2e/playwright/app-header-tests.spec.ts
View File

@ -22,13 +22,13 @@ test.describe('Electron app header tests', () => {
let text
switch (process.platform) {
case 'darwin':
text = 'Commands⌘ K'
text = 'Commands⌘K'
break
case 'win32':
text = 'CommandsCtrl + K'
text = 'CommandsCtrl+K'
break
default: // 'linux' etc.
text = 'CommandsCtrl K'
text = 'CommandsCtrl+K'
break
}
const commandsButton = page.getByRole('button', { name: 'Commands' })

23
src/lib/hotkeyWrapper.ts
View File

@ -47,9 +47,18 @@ export function hotkeyDisplay(hotkey: string, platform: Platform): string {
const isMac = platform === 'macos'
const isWindows = platform === 'windows'
const meta = isWindows ? 'Win' : 'Meta'
const separator = isWindows ? ' + ' : ' '
const outputSeparator = isMac ? '' : '+'
const display = hotkey
.replaceAll('+', separator)
// Capitalize letters. We want Ctrl K, not Ctrl k, since Shift should be
// shown as a separate modifier.
.split('+')
.map((word) => {
if (word.length === 1 && LOWER_CASE_LETTER.test(word)) {
return word.toUpperCase()
}
return word
})
.join(outputSeparator)
// Collapse multiple spaces into one.
.replaceAll(WHITESPACE, ' ')
.replaceAll('mod', isMac ? '⌘' : 'Ctrl')
@ -63,16 +72,6 @@ export function hotkeyDisplay(hotkey: string, platform: Platform): string {
// The correct arrow is ⇧ "UPWARDS WHITE ARROW" Unicode: U+21E7
.replaceAll('shift', isMac ? '⬆' : 'Shift')
.replaceAll('alt', isMac ? '⌥' : 'Alt')
// Capitalize letters. We want Ctrl K, not Ctrl k, since Shift should be
// shown as a separate modifier.
.split(separator)
.map((word) => {
if (word.length === 1 && LOWER_CASE_LETTER.test(word)) {
return word.toUpperCase()
}
return word
})
.join(separator)
return display
}