You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: backlog/completed/task-260 - Fix-keyboard-shortcut-collision-with-alphabetical-artist-navigation-in-library.md
+35-9Lines changed: 35 additions & 9 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -1,9 +1,10 @@
1
1
---
2
-
id: task-260
2
+
id: TASK-260
3
3
title: Fix keyboard shortcut collision with alphabetical artist navigation in library
4
-
status: To Do
4
+
status: Done
5
5
assignee: []
6
6
created_date: '2026-02-06 23:13'
7
+
updated_date: '2026-02-07 04:03'
7
8
labels:
8
9
- frontend
9
10
- ux
@@ -12,6 +13,7 @@ labels:
12
13
dependencies:
13
14
- task-107
14
15
priority: medium
16
+
ordinal: 500
15
17
---
16
18
17
19
## Description
@@ -34,11 +36,35 @@ Both approaches should be evaluated during implementation planning. The chosen s
34
36
35
37
## Acceptance Criteria
36
38
<!-- AC:BEGIN -->
37
-
-[]#1 Users can press letter keys (A-Z) to jump to artists starting with that letter in the library browser
-[]#3 No ambiguity exists between alphabetical navigation and shortcut activation -- the system deterministically picks one behavior based on clear rules
40
-
-[]#4 Existing shortcut behavior from task-107 is not broken (Space, arrow keys, Cmd+F, Cmd+D, Cmd+S, Escape, Delete)
41
-
-[]#5 Focus state or modifier key requirement is clearly communicated to the user (e.g., visual focus ring on library, shortcut hints in settings)
42
-
-[]#6 Alphabetical jump scrolls the library list to the first artist matching the pressed letter
43
-
-[]#7 Pressing the same letter again cycles to the next artist starting with that letter
39
+
-[x]#1 Users can press letter keys (A-Z) to jump to artists starting with that letter in the library browser
-[x]#3 No ambiguity exists between alphabetical navigation and shortcut activation -- the system deterministically picks one behavior based on clear rules
42
+
-[x]#4 Existing shortcut behavior from task-107 is not broken (Space, arrow keys, Cmd+F, Cmd+D, Cmd+S, Escape, Delete)
43
+
-[x]#5 Focus state or modifier key requirement is clearly communicated to the user (e.g., visual focus ring on library, shortcut hints in settings)
44
+
-[x]#6 Alphabetical jump scrolls the library list to the first artist matching the pressed letter
45
+
-[x]#7 Pressing the same letter again cycles to the next artist starting with that letter
44
46
<!-- AC:END -->
47
+
48
+
## Final Summary
49
+
50
+
<!-- SECTION:FINAL_SUMMARY:BEGIN -->
51
+
Implemented Option A (modifier keys) to resolve the keyboard shortcut collision with alphabetical artist navigation.
52
+
53
+
**Changes:**
54
+
55
+
1.**shortcuts.js**: Moved single-letter shortcuts to modifier combos:
- Updated SHORTCUT_DEFINITIONS for settings display
61
+
62
+
2.**library-browser.js**: Added same-letter cycling for type-to-jump (AC#7):
63
+
- Pressing the same letter repeatedly cycles through distinct matching artists
64
+
- Wraps around after the last match
65
+
- Cycling state resets on debounce timeout or different character input
66
+
67
+
3.**settings.html**: Added tip text explaining type-to-jump and cycling behavior
68
+
69
+
4.**Tests**: 16 new Vitest unit tests for modifier shortcuts, 2 new Playwright E2E tests for cycling behavior. All 246 unit tests and 630 E2E tests pass.
0 commit comments