Skip to content

Improve search page styling with dark mode enabled#13488

Merged
sfshaza2 merged 2 commits into
mainfrom
fix/13336
Jun 10, 2026
Merged

Improve search page styling with dark mode enabled#13488
sfshaza2 merged 2 commits into
mainfrom
fix/13336

Conversation

@parlough

@parlough parlough commented Jun 10, 2026

Copy link
Copy Markdown
Member

Overrides additional styles to improve styling of GSC results in dark mode with some smaller benefits in light mode as well.

Fixes #13336

Staged: https://flutter-docs-prod--docs-pr13488-fix-13336-4g95g227.web.app/search?q=testing

@parlough parlough requested review from a team and sfshaza2 as code owners June 10, 2026 12:28

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the search styling in _search.scss by setting the text color of the search input, adding a top margin to the completion box, and removing backgrounds for search results and cursor pages. The reviewer suggests appending !important to the new color property on the search input to ensure it consistently overrides styles across different browsers and themes.

Comment thread sites/docs/lib/_sass/pages/_search.scss
@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit e083d61):

https://flutter-docs-prod--docs-pr13488-fix-13336-4g95g227.web.app

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated flutter.dev site (updated for commit e083d61):

https://flutter-dev-230821--www-pr13488-fix-13336-uszeye34.web.app

@sfshaza2 sfshaza2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@sfshaza2 sfshaza2 merged commit a007b99 into main Jun 10, 2026
13 checks passed
@sfshaza2 sfshaza2 deleted the fix/13336 branch June 10, 2026 18:15
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.

Site search page results dark mode issues

3 participants