Skip to content

Commit 6d6d163

Browse files
authored
Fix keyboard shortcut modifiers (removing redundant identity replacement) (#597)
fix for https://github.com/SableClient/Sable/security/code-scanning/1 removed the redundant replacement with it self. #### Type of change - [x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update ### Checklist: - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings ### AI disclosure: - [ ] Partially AI assisted (clarify which code was AI assisted and briefly explain what it does). - [ ] Fully AI generated (explain what all the generated code does in moderate detail). codeql was used, if that counts, but it was a one line change, so no AI was used
2 parents 63afb74 + 7213aff commit 6d6d163

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

src/app/features/settings/keyboard-shortcuts/KeyboardShortcuts.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,7 @@ function formatKey(key: string): string {
2323
return key
2424
.replace(/\bmod\b/g, isMac ? '⌘' : 'Ctrl')
2525
.replace(/\balt\b/gi, isMac ? '⌥' : 'Alt')
26-
.replace(/\bshift\b/gi, '⇧')
27-
.replace(/\+/g, '+');
26+
.replace(/\bshift\b/gi, '⇧');
2827
}
2928

3029
const SHORTCUT_CATEGORIES: ShortcutCategory[] = [

0 commit comments

Comments
 (0)