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

The site search box is named in docs/design/areas/search.md, and .search leaves §5d #5573

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

This adds `docs/design/areas/search.md`, which names `.search` as the search field of a filter or search form. The field grows to fill its row, from 180px to 320px wide. The file lists every place that writes the class: - `searchPage` in `src/web/pages/search.ts` (the `/search` form) - the filter forms in `src/web/pages/browse.ts`, `org.ts` and `origins.ts` - `src/web/studio/explore.ts`

shipped production is serving it deploy 2fa0c24

opened this

This adds docs/design/areas/search.md, which names .search as the search field of a filter or search form. The field grows to fill its row, from 180px to 320px wide. The file lists every place that writes the class:

  • searchPage in src/web/pages/search.ts (the /search form)
  • the filter forms in src/web/pages/browse.ts, org.ts and origins.ts
  • src/web/studio/explore.ts

It also names the two rules that style it, .filters .search and .bar .search, in src/web/style/content.ts. The one - .search` line is deleted from §5d of docs/DESIGN.md`, and nothing else there changes. No markup or stylesheet changed, so every page looks the same.

The issue says the top bar uses this component, but it doesn't: src/web/layout.ts writes .topbar-search. I left .topbar-search in §5d because the issue's class list names only .search. A follow-up can move it into this file if it belongs here.

bun scripts/design-lint.ts passes, and so do test/design-lint.test.ts and test/review-design.test.ts.

🤖 Generated with Claude Code

Closes #5540

commented

This adds docs/design/areas/search.md, which names .search as the search field of a filter or search form. The field grows to fill its row, from 180px to 320px wide. The file lists every place that writes the class:

  • searchPage in src/web/pages/search.ts (the /search form)
  • the filter forms in src/web/pages/browse.ts, org.ts and origins.ts
  • src/web/studio/explore.ts

It also names the two rules that style it, .filters .search and .bar .search, in src/web/style/content.ts. The one - .search` line is deleted from §5d of docs/DESIGN.md`, and nothing else there changes. No markup or stylesheet changed, so every page looks the same.

The issue says the top bar uses this component, but it doesn't: src/web/layout.ts writes .topbar-search. I left .topbar-search in §5d because the issue's class list names only .search. A follow-up can move it into this file if it belongs here.

bun scripts/design-lint.ts passes, and so do test/design-lint.test.ts and test/review-design.test.ts.

🤖 Generated with Claude Code

Approved

Docs-only and accurate: I checked that every file the new page lists writes class "input search" (search.ts:366, browse.ts:176, org.ts:106, origins.ts:1396, explore.ts:108), and that content.ts:173 and :179 hold the .filters and .bar rules with 180px/320px. The §5d line is removed and nothing else moves; no markup or CSS changed, so no preview impact. Next: (1) move .topbar-search into a doc like this, as the author suggests; (2) add the screen-reader name (aria-label) for the field to the entry, since a search box without a label is the usual a11y miss.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs