Skip to content

Commit 5dca3bf

Browse files
committed
refactor(ui): extract named booleans for Ctrl+N/P shortcut conditions
1 parent cd55515 commit 5dca3bf

File tree

1 file changed

+6
-16
lines changed

1 file changed

+6
-16
lines changed

app/components/CommandPalette.client.vue

Lines changed: 6 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -173,14 +173,11 @@ function handleGlobalKeydown(event: KeyboardEvent) {
173173
174174
if (!isOpen.value) return
175175
176-
if (
177-
event.key === 'ArrowDown' ||
178-
(event.ctrlKey &&
179-
!event.altKey &&
180-
!event.metaKey &&
181-
!event.shiftKey &&
182-
event.key.toLowerCase() === 'n')
183-
) {
176+
const isCtrlOnly = event.ctrlKey && !event.altKey && !event.metaKey && !event.shiftKey
177+
const isNextShortcut = isCtrlOnly && event.key.toLowerCase() === 'n'
178+
const isPrevShortcut = isCtrlOnly && event.key.toLowerCase() === 'p'
179+
180+
if (event.key === 'ArrowDown' || isNextShortcut) {
184181
event.preventDefault()
185182
const currentIndex = getCommandElements().findIndex(el => el === document.activeElement)
186183
const nextIndex =
@@ -189,14 +186,7 @@ function handleGlobalKeydown(event: KeyboardEvent) {
189186
return
190187
}
191188
192-
if (
193-
event.key === 'ArrowUp' ||
194-
(event.ctrlKey &&
195-
!event.altKey &&
196-
!event.metaKey &&
197-
!event.shiftKey &&
198-
event.key.toLowerCase() === 'p')
199-
) {
189+
if (event.key === 'ArrowUp' || isPrevShortcut) {
200190
event.preventDefault()
201191
const currentIndex = getCommandElements().findIndex(el => el === document.activeElement)
202192
if (currentIndex <= 0) {

0 commit comments

Comments
 (0)