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

.filters leaves src/web/style/content.ts and docs/DESIGN.md §5d #4688

Closed opened this
opened this

Done:

  • [ ] Every .filters (and .filters .search, .filters .seg) rules are deleted from src/web/style/ and from docs/DESIGN.md §5d, and nothing else in those files changes.
  • [ ] grep -rn finds none of them under src/; bun scripts/design-lint.ts is clean and bun test test/design-lint.test.ts passes.

Files: src/web/style/*.ts, docs/DESIGN.md, docs/design/baseline.json only. It waits for the /search markup issue, which takes the classes out of the markup. Rebase on main just before pushing and check grep -n '^<<<<<<<\|^>>>>>>>' src/web/style finds nothing: the last try at this kind of pull failed on committed conflict markers.

commented

Parked by the crew: the run ended without a pull: declined: I can't do this yet without changing how /explore looks. Every issue #4688 waits for is closed, but src/web/studio/explore.ts:118 on main (05d93e01) still renders <div class="filters mb"> around t

commented

Parked by the crew: the check failed ("the stylesheet carries no merge-conflict markers") and the decider saw no point in another try (pull #4799 closed).

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

  1. the run ended without a pull: declined: I can't do this yet without changing how /explore looks. Every issue #4688 waits for is closed, but src/web/studio/explore.ts:118 on main (05d93e01) still renders <div class="filters mb"> around t
  2. the check failed ("the stylesheet carries no merge-conflict markers") and the decider saw no point in another try (pull #4799 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 #6241. Both parks are behind us: the markup no longer renders .filters anywhere on main (the /explore and /search forms draw §5's .bar, and test/filters-markup.test.ts proves it) and docs/DESIGN.md §5d no longer names it. Only the seven CSS rules in src/web/style/content.ts and one sentence in docs/design/areas/search.md are left, and #6241 says exactly that, with the conflict-marker check before the push.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs