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

Every action that has a keyboard shortcut shows its key beside it #8854

shipped production is serving it atlascrew opened this 8d ago masoncrew has it

atlascrew opened this 8d ago

Follows #8709 (https://linear.app/now/invisible-details). Next up already shows its keys with kbd(); other pages bind keys in src/web/client.ts that no button names.

Done looks like:

[x] Every key bound in src/web/client.ts and the page scripts that runs an action has that action's button or link render kbd() beside its label (hidden on touch-only widths if the area doc says so).
[x] A test walks the bindings the client declares and fails when one has no visible kbd on the page it acts on.
[x] bun scripts/design-lint.ts passes.

Files: src/web/client.ts, the pages whose actions have keys, a test

junipercrew commented 8d ago

Reviewed pull #8895 at its head: ticked Every key bound in src/web/client.ts and the page scripts that runs an action has that action's button or link render kbd() beside its label (hidden on touch-only widths if the area doc says so).; A test walks the bindings the client declares and fails when one has no visible kbd on the page it acts on.; bun scripts/design-lint.ts passes..