Skip to content

Commit eb46df9

Browse files
enkoclaude
andcommitted
fix(frontend): Improve keyboard filter UX with modal and auto-scroll
- Convert filter dropdown to centered modal when opened via keyboard - Add auto-scroll to selected category section in modal - Fix Escape key to close filter modal - Fix "shift..." display bug by checking keyLower.length === 1 - Remove nested scroll containers to fix circles scrolling issue Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent ced60ec commit eb46df9

File tree

2 files changed

+263
-36
lines changed

2 files changed

+263
-36
lines changed

apps/frontend/src/lib/components/KeyboardShortcuts.svelte

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -284,7 +284,13 @@ function handleKeydown(e: KeyboardEvent) {
284284
}
285285
286286
// Check if it's a letter for extended navigation (not a category key)
287-
if (keyLower >= 'a' && keyLower <= 'z' && !FILTER_CATEGORY_KEYS[keyLower]) {
287+
// Must be single character to avoid capturing modifier keys like "shift"
288+
if (
289+
keyLower.length === 1 &&
290+
keyLower >= 'a' &&
291+
keyLower <= 'z' &&
292+
!FILTER_CATEGORY_KEYS[keyLower]
293+
) {
288294
e.preventDefault();
289295
filterModePrefix.set(keyLower);
290296
return;

0 commit comments

Comments
 (0)