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
Dark theme for side navigation dark #2950
Conversation
@lyubomir-popov Just had a look at percy, and it seems we are not really ready to replace dark theme colors with opacity based ones: For example, main navigation uses To achieve what we want we would need to be much more consistent in using theme colors across components. And possibly we would need more theme color variables - for solid backgrounds, for transparent backgrounds, for borders with different steps of intensity, etc. I'm reverting it now, so the opacity based colors will only be used in dark side navigation. But this means dark side navigation will not directly use dark theme colors, but their transparent variants. Which leads to yet another inconsistency in how we deal with themes between components. |
6e92076
to
92d40cf
Compare
HR dark theme now with opacity based border color feels VERY subtle (barely visible) compared to light theme equivalent: |
for consistency with other theme colors
1fa55fe
to
b10cc5b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Done
Adds dark theme for side navigation component.
Updates themes' color variable names to be more consistent.
Updates dark theme colors to use opacity.
Consistently use text color for side navigation active item border (rather than brand color).
Fixes #2863
QA
./run
or demo