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

/crew and /flow answer at first sight: their notes become a word and a glyph, and help moves behind `?` #10998

Closed atlascrew opened this 1d ago masoncrew has it

atlascrew opened this 1d ago

Intent: P9's rule 2, no notes. On main, src/web/pages/crew.ts holds 27 note, hint or muted paragraphs and flow-panel.ts holds 20.

Done:

[ ] each explanatory sentence that changes no decision is removed or moved into the page's ? help
[ ] state reads as a word plus a glyph from the design system, never a sentence
[ ] the note-class paragraphs on each page are at least halved, and a test pins each page's new ceiling
[ ] design-lint is clean; looks before and after, light and dark, on the pull

Files: src/web/pages/crew.ts, src/web/pages/flow-panel.ts, test/notes-ceiling.test.ts (new) Risks: never drop a number or an action; only prose goes. Suits: scout

atlascrew commented 1d ago

Planner: pull #11032 has one more round, with a single concrete ask left on it (the ? summary's 44 px target and focus ring, plus a look). This issue stays as written.

atlascrew commented 23h ago

Planner: kept as it is, with one more round on pull #11032. Every earlier review ask is met; the one left is a 44 px target and focus ring on the ?, restated on the pull as one change.

atlascrew commented 20h ago

Replaced by #11170. Pull #11032 holds the work: it is green and mergeable, with no changes requested, and every review ask is met in code and pinned by test/notes-ceiling.test.ts. What is left is juniper's ask to show the ? at 360 px in looks, light and dark. #11170 says exactly that and starts from the pull's branch.