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

.merge-box, .merge-form, .merge-headline and .merge-sub leave the stylesheet and docs/DESIGN.md §5d #4400

Closed opened this
opened this

Part three of #3867.

Done:

  • [ ] git grep -n "merge-box\|merge-form\|merge-headline\|merge-sub" src finds only their rules in src/web/style/content.ts. If one is still used, keep it and say where in the pull.
  • [ ] The rules (and any reduced-motion or narrow-screen block for them) leave src/web/style/content.ts, and the four names leave their §5d bullet in docs/DESIGN.md.
  • [ ] Run bun scripts/design-lint.ts --update. Commit docs/design/baseline.json only if the count goes down or stays equal.
  • [ ] bun x tsc --noEmit and bun test test/design-lint.test.ts pass.
commented

Parked by the crew: the check failed ("spend that cannot be taken back > nor does deleting a run, removing an app, or deleting the organisation") and the decider saw no point in another try (pull #4438 closed). 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 failed ("the index carries the hot queries > (unnamed)") and the decider saw no point in another try (pull #4513 closed).

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

  1. the check failed ("spend that cannot be taken back > nor does deleting a run, removing an app, or deleting the organisation") and the decider saw no point in another try (pull #4438 closed)
  2. the check failed ("the index carries the hot queries > (unnamed)") and the decider saw no point in another try (pull #4513 closed)

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 #4609. Both tries went red on tests the change never touched (migrate-after, origins-reconnect, livetriage, snapshot-mirror, strangers, or an "(unnamed)" hook timeout); those are now planned as #4577–#4582. The fresh issue keeps the diff to the named files and says to run design-lint and lower the baseline in the same pull.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs