header search bar (Cmd+K) + dialog redesign — cherry-pick to main - #698
Merged
Conversation
Bundles the search-experience work merged on dev into a single
cherry-pick to main. Includes:
- Cmd+K / Ctrl+K / '/' to open; ESC to close
- Wide input-pill trigger in the header (Linear-style, no shortcut badge)
- cmdk + Fuse.js fuzzy search over ~400-doc index (benchmarks, products,
alternatives, answers, chains, compare pairs, pages)
- Modal redesign: hairline border, single shadow, no accent focus ring
(opt-out from the global :focus-visible accent rule)
- Idle state: Recent (localStorage chips, X-to-remove, Clear) + Live
leaders (horizontal snap cards with leader logo + p50 from /api/citable)
+ Trending list (vertical, same data shape)
- Search state: results grouped by kind, each row gets the leader's
logo (benches) / entity logo (products/chains) / kind icon fallback
- Mobile: full-bleed sheet, footer hidden, X close button
- Open animation: handcrafted CSS keyframes (fade + 4px slide-down,
150ms ease-out), reduced-motion fallback
Template fix bundled (search-adjacent — surfaces description tokens):
- renderBenchmarkText now resolves seoTitle / seoDescription / subtitle /
methodology / disclaimer (Featured / Trending cards in the search
dialog were showing raw '{{best_name}}' tokens in description previews)
- Chain stash lookup case-insensitive (fixes perp-fees uppercase chain
values ETH/BTC/SOL)
- Bumped bench-unfiltered v11→v12 + all-benchmarks v15→v16 so the
cached snapshots regenerate through the wider resolver instead of
serving raw tokens for up to 300s after deploy
Deps:
- cmdk@^1.1.1, fuse.js@^7.4.2 (lockfile synced from dev)
Header redesign:
- Removed | separator between Contribute and GitHub
- 15px nav, h-16 desktop, backdrop-blur on the sticky
- GitHub icon-only, gap-spaced utilities
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cherry-pick of the search-bar work shipped on dev (PRs #671, #677, #680, #682, #687, #688) into a single focused PR to main.
What lands
Template fix bundled (search-adjacent)
renderBenchmarkTextnow resolvesseoTitle/seoDescription/subtitle/methodology/disclaimer— the Featured / Trending cards in the dialog were showing raw{{best_name}}in description previews.bench-unfiltered v11→v12+all-benchmarks v15→v16so cached snapshots regenerate immediately on deploy.Deps
Scope kept tight
Did NOT bring dev's unrelated spec.ts refactors (quorum guard, MS_PER_MINUTE, expectedN overlay). Cache keys bumped from main's baseline (v11→v12, v15→v16), not dev's (v13/v17).
Test plan
{{best_name}}in any description.