Skip to content

Search filter polish: wave colors, screen-pinned close, archive year - #161

Merged
RonanHevenor merged 1 commit into
mainfrom
fiddle-with-the-search-bar
Sep 17, 2026
Merged

RonanHevenor merged 1 commit into
mainfrom
fiddle-with-the-search-bar

Conversation

@RonanHevenor

Copy link
Copy Markdown
Member

Follow-ups on the search filters, all from testing the live overlay.

  • Filter words use the header wave's own colors. ThemeStyle publishes waveColor1/2/3 from the theme as --wave-color-1/2/3, so the effect follows whatever the newsroom sets (blue today) instead of a hardcoded rainbow.
  • Effect timing. It fades in half a second after the wave settles rather than running during it, and fades out smoothly instead of snapping. Changing a filter runs the wave bar only — the words stay put.
  • Picking an option no longer leaves the word stuck in the accent color (the hidden <select> kept focus).
  • Close control is pinned to the screen, not the content column — it was inside a transformed element, which makes fixed resolve against that element. Bigger X in #D6001C with a gray esc key hint beside it (hidden on phones). Escape still closes; background clicks no longer do.
  • /api/search/archive-date returns 2001. The legacy Wagtail MIN(first_published_at) is an import date, so it claimed 2018 while search actually reaches back to 2001. Now a constant, which also drops a legacy DB connection per call.

🤖 Generated with Claude Code

…ve year

- Filter words catch the header wave's own colors (ThemeStyle now publishes
  waveColor1/2/3 as CSS variables), fading in and out half a second after the
  wave settles, and skipped entirely when only a filter changed
- Picking an option no longer leaves the word stuck in the accent color
- Close control pinned to the screen instead of the content column (it sat
  inside a transformed element), bigger X in #D6001C with an "esc" key hint;
  Escape still closes, background clicks no longer do
- /api/search/archive-date returns 2001: the legacy Wagtail
  MIN(first_published_at) reports import dates (2018), not coverage

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@RonanHevenor
RonanHevenor merged commit f2a2c8c into main Sep 17, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant