Skip to content

fix: pin @docsearch/css to 3.9.0 to match @docsearch/react v3#17878

Merged
pettinarip merged 1 commit into
stagingfrom
fix/docsearch-css-version-mismatch
Mar 27, 2026
Merged

fix: pin @docsearch/css to 3.9.0 to match @docsearch/react v3#17878
pettinarip merged 1 commit into
stagingfrom
fix/docsearch-css-version-mismatch

Conversation

@pettinarip
Copy link
Copy Markdown
Member

@pettinarip pettinarip commented Mar 27, 2026

Summary

What was broken

  1. Search icon in primary/purple color — v4 CSS doesn't override the button's inherited text-primary, while v3 sets body color on .DocSearch-Search-Icon
  2. Constant spinner in search modal — v4 CSS shows .DocSearch-LoadingIndicator by default (display: flex), while v3 hides it unless .DocSearch-Container--Stalled is present
  3. Hover/highlight style issues — v4 CSS variables and selectors don't align with the v3 React component's DOM structure

Reported in #17874 (comment): #17874 (comment)

Test plan

  • Search icon in nav should be body color (dark), not purple
  • Opening search modal should show no spinner when idle
  • Search results should have correct hover and highlight colors

PR #17818 added @docsearch/css@^4.6.0 as an explicit dependency, but
@docsearch/react is still at v3.9.0 which expects @docsearch/css@3.9.0.
The v4 CSS caused several regressions: purple search icon, constant
spinner in modal, and broken hover/highlight styles.
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 27, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit e54ac67
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69c6932a12a5650008c4035e
😎 Deploy Preview https://deploy-preview-17878.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 57 (🟢 up 5 from production)
Accessibility: 94 (no change from production)
Best Practices: 100 (no change from production)
SEO: 98 (🔴 down 1 from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the dependencies 📦 Changes related to project dependencies label Mar 27, 2026
@pettinarip pettinarip changed the base branch from dev to staging March 27, 2026 14:25
@pettinarip pettinarip mentioned this pull request Mar 27, 2026
@pettinarip pettinarip merged commit 965b5eb into staging Mar 27, 2026
6 of 7 checks passed
@pettinarip pettinarip deleted the fix/docsearch-css-version-mismatch branch March 27, 2026 14:32
@wackerow
Copy link
Copy Markdown
Member

Thanks @pettinarip, yeah would've been from #17818 🙏

@pettinarip pettinarip mentioned this pull request Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies 📦 Changes related to project dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants