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

.front-admin and .front-pick leave the stylesheet #5065

Closed opened this
opened this

Intent: the front page's admin controls use §5's pieces. .front-admin (a centred max-width wrapper) and .front-pick (a row gap and an inline form margin) are each used once in src/web/studio/home.ts (near lines 416 and 561) and defined in src/web/style/content.ts:313-315.

Done:

  • [ ] both draw from base.ts utilities or §5's row and bar, so the page looks the same at 360px and 1280px (before and after looks in the pull)
  • [ ] the rules leave content.ts and the names leave docs/DESIGN.md §5d
  • [ ] bun scripts/design-lint.ts and the home's tests pass
commented

Parked by the crew: the check failed ("test/spawn-child-runs-apps.test.ts > a hosted child's apps run in the parent > (unnamed)") and the decider saw no point in another try (pull #5086 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 ("apps: production, previews, data, gates > require_checks holds the merge until the head's deploy is green") and the decider saw no point in another try (pull #5188 closed).

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

  1. the check failed ("test/spawn-child-runs-apps.test.ts > a hosted child's apps run in the parent > (unnamed)") and the decider saw no point in another try (pull #5086 closed)
  2. the check failed ("apps: production, previews, data, gates > require_checks holds the merge until the head's deploy is green") and the decider saw no point in another try (pull #5188 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 #5217: same scope, but it waits for fixes to the two unrelated tests that failed its tries (#5203, #5204).

commented

Closed as dropped, not split: the forge refused a one-for-one split here. Nothing is lost. #5217 carries this scope word for word, under the same pillar, and waits for the fixes to the unrelated red tests.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs