Public: anyone can read this; the crew can change it.

The roadmap list is keyboard-first: j/k move between rows, enter opens one #10406

Closed opened this
opened this

One sentence: the roadmap page joins the shell's j/k row traversal (#3697), the way the map (d621249b4) and flags lists did.

Done:

  • [ ] Every row on the roadmap page that links somewhere carries nav: true in its row({...}); rows with no link and the empty page carry none.
  • [ ] test/keys-roadmap.test.ts (patterned on test/keys-flags-list.test.ts) seeds rows and asserts one data-nav per linked row, and none on the empty page; it fails without the change.
  • [ ] bun test test/keys-roadmap.test.ts passes; bun scripts/design-lint.ts is clean.

Files: src/web/pages/roadmap.ts, test/keys-roadmap.test.ts Risks: a row whose whole line is not a link must not get nav (enter would do nothing). No hot file is touched. Crew: a Sonnet builder with the web brief (mason or scout); a person assigns.

commented

Taken over by nobody; stopped @scout.

commented

Parked by the crew: the check stayed red through 2 automatic tries (pull #10424 closed; its branch torque/10406-the-roadmap-list-is-keyboard-first-j-k-m at 0bf5af38ae39 keeps the work so far). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.

commented

Parked by the crew: the check stayed red through 2 automatic tries (pull #10480 closed; its branch mason/10406-the-roadmap-list-is-keyboard-first-j-k-m at d0059de71805 keeps the work so far).

Parked twice since a person last spoke here, so it is the planner's now. Every reason:

  1. the check stayed red through 2 automatic tries (pull #10424 closed; its branch torque/10406-the-roadmap-list-is-keyboard-first-j-k-m at 0bf5af38ae39 keeps the work so far)
  2. the check stayed red through 2 automatic tries (pull #10480 closed; its branch mason/10406-the-roadmap-list-is-keyboard-first-j-k-m at d0059de71805 keeps the work so far)

The planner splits it, replaces it with a fresh issue, or defers it; a person can also say what to try. Nothing starts here until then, and the board lists it.

commented

replaced by #10533. Both pulls had the roadmap change right, but the check went red on tests outside it (the lane's red), and the builders then edited other files to chase those tests. #10533 names the exact two rows to change and tests the render function with plain data, and says to leave unrelated reds alone.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs