feat(board): keyboard navigation + letter shortcuts (B4)

dev commented

2026-04-27 00:39:27 +00:00

Collaborator

Adds full keyboard navigation to the Kanban board so operators can triage without touching the mouse.

Test plan

j / k moves focus between cards within a column; h / l moves between columns. Focused card shows a 2 px outline-accent ring distinct from the B3 selection ring.
Focus initialises on mount to the first card of the first non-empty column and survives SSE-driven cache patches.
a on a focused card opens the agent-picker dialog; selecting an agent (or Unassign) calls POST /board/assign.
s shows a toast "Stage change requires B2 — not yet available".
r opens the reroute-confirm dialog; confirming calls POST /board/reroute and posts an audit comment.
? opens the keymap overlay; Esc closes it.
Esc without an open overlay clears the B3 side-panel selection.
All shortcuts are suppressed when focus is in a filter input or textarea.
All shortcuts except Esc are suppressed while an overlay is open.
just qa clean; 447 tests pass.

Closes #412

Adds full keyboard navigation to the Kanban board so operators can triage without touching the mouse. ## Test plan - [ ] `j` / `k` moves focus between cards within a column; `h` / `l` moves between columns. Focused card shows a 2 px `outline-accent` ring distinct from the B3 selection ring. - [ ] Focus initialises on mount to the first card of the first non-empty column and survives SSE-driven cache patches. - [ ] `a` on a focused card opens the agent-picker dialog; selecting an agent (or Unassign) calls `POST /board/assign`. - [ ] `s` shows a toast "Stage change requires B2 — not yet available". - [ ] `r` opens the reroute-confirm dialog; confirming calls `POST /board/reroute` and posts an audit comment. - [ ] `?` opens the keymap overlay; `Esc` closes it. - [ ] `Esc` without an open overlay clears the B3 side-panel selection. - [ ] All shortcuts are suppressed when focus is in a filter input or textarea. - [ ] All shortcuts except `Esc` are suppressed while an overlay is open. - [ ] `just qa` clean; 447 tests pass. Closes #412

dev added 1 commit

2026-04-27 00:39:27 +00:00

feat(board): j/k card focus + a/s/r letter shortcuts + ? keymap overlay

qa / qa (pull_request) Successful in 5m38s

Details

qa / dockerfile (pull_request) Successful in 9s

Details

5e6076ad12

Adds full keyboard navigation to the Kanban board (B4 / #412):

- j/k — focus next/prev card in column; h/l — focus left/right column.
  Focus persists across SSE-driven cache patches; initialised on mount to
  the first card of the first non-empty column.
- Focused card shows a 2 px outline-accent ring (outline-offset-2),
  distinct from the selection ring.
- a — agent-picker dialog (lists all board columns + Unassign).
  Unassign path wires null targetType through the assign mutation.
- s — stage-picker placeholder toast until B2 lands.
- r — reroute-confirm dialog (cancels running task, audit comment).
  Adds rerouteMutation to planner.board route.
- ? — keymap overlay (Navigation / Card actions / Selection groups).
- Esc — closes any open overlay first, then falls through to B3 selection.
- All shortcuts suppressed in text inputs/textareas/contenteditable.
- All shortcuts except Esc suppressed while an overlay is open.

Tests (5 new in board.test.tsx):
  j/k cycles within a column; h/l moves between columns;
  a opens picker and calling onAssign with focused card;
  ? opens overlay and Esc dismisses; suppressed inside filter inputs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

dev requested review from reviewer

2026-04-27 00:51:09 +00:00

reviewer approved these changes

2026-04-27 00:53:42 +00:00

reviewer left a comment

CI green (run #2224). All five required B4 tests pass. AC coverage solid.

Nit: when the focused card disappears due to an SSE patch, board.tsx sets focusedKey(null) and the init effect re-fires, landing on the first card of the first non-empty column. Issue #412 AC says "focus moves to the next card in the column, or up to the column header if empty." The behaviour is reasonable but doesn't match the spec exactly — worth a follow-up but not a blocker given it's a rare event and both paths leave focus somewhere sensible.

CI green (run #2224). All five required B4 tests pass. AC coverage solid. Nit: when the focused card disappears due to an SSE patch, `board.tsx` sets `focusedKey(null)` and the init effect re-fires, landing on the first card of the first non-empty column. Issue #412 AC says "focus moves to the next card in the column, or up to the column header if empty." The behaviour is reasonable but doesn't match the spec exactly — worth a follow-up but not a blocker given it's a rare event and both paths leave focus somewhere sensible.

dev force-pushed dev/412 from 5e6076ad12

qa / qa (pull_request) Successful in 5m38s

Details

qa / dockerfile (pull_request) Successful in 9s

Details

to 705f6a0cff

qa / qa (pull_request) Successful in 6m16s

Details

qa / dockerfile (pull_request) Successful in 9s

Details

2026-04-27 01:20:49 +00:00

Compare