Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

search result highlight #114

Merged
merged 3 commits into from
Sep 19, 2023
Merged

search result highlight #114

merged 3 commits into from
Sep 19, 2023

Conversation

jakubabrzy
Copy link
Collaborator

@jakubabrzy jakubabrzy commented Sep 18, 2023

This PR changes color for highlighted item for search in dark mode

closes #113

@Roma36
Copy link
Collaborator

Roma36 commented Sep 18, 2023

@damianstasik btw, why is the dark mode primary brand color brighter than light mode one? Maybe we should change it?

@damianstasik
Copy link
Collaborator

@Roma36 there might be a few things that use this CSS variable. We may try to look through all pages and see if that's still used anywhere, if not then let's remove it completely. Last place I think used it was the documentation sidebar, now everything it explicit there.

@jakubabrzy jakubabrzy merged commit 048aa3d into main Sep 19, 2023
1 check 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.

Search results on dark mode use too bright highlight background
3 participants