Skip to content

Add keyboard nav (arrows + space) to accordion pane picker - #313

Merged
komacke merged 2 commits into
openhamclock:mainfrom
wasatch-dev:main
Aug 9, 2026
Merged

Add keyboard nav (arrows + space) to accordion pane picker#313
komacke merged 2 commits into
openhamclock:mainfrom
wasatch-dev:main

Conversation

@wasatch-dev

Copy link
Copy Markdown
Collaborator
  • plotmgmnt.cpp: accordionKbMove() moves focus among the current frame's visible rows (headers + expanded children) using the same nearest-neighbor-in-direction logic as menu.cpp's kbNavigation()
  • Space activates the focused row (toggle item, or expand/collapse category); Enter triggers Ok (same "Select an item" gating as tap); Esc already cancelled
  • Focus highlight reuses menuDrawItem()'s existing kb_focus green border; no highlight shown until an arrow key is first pressed
  • Note: opening the picker itself via keyboard alone still relies on the existing want_kbcursor/-y cursor-warp feature (touch.cpp), unrelated to this change

- plotmgmnt.cpp: accordionKbMove() moves focus among the current
  frame's visible rows (headers + expanded children) using the same
  nearest-neighbor-in-direction logic as menu.cpp's kbNavigation()
- Space activates the focused row (toggle item, or expand/collapse
  category); Enter triggers Ok (same "Select an item" gating as tap);
  Esc already cancelled
- Focus highlight reuses menuDrawItem()'s existing kb_focus green
  border; no highlight shown until an arrow key is first pressed
- Note: opening the picker itself via keyboard alone still relies on
  the existing want_kbcursor/-y cursor-warp feature (touch.cpp),
  unrelated to this change
…or IOTA island name and identifier on clicking either frequency in flex pane or when clicking on cyan colored I in small, upper pane. Requires backend iota.txt for full experience
@komacke
komacke merged commit 7410e2c into openhamclock:main Aug 9, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants