Skip to content

Add dark mode css variables #2802

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

Merged
merged 2 commits into from
Sep 13, 2024
Merged

Conversation

wcole3
Copy link
Contributor

@wcole3 wcole3 commented Sep 11, 2024

Mostly opening for discussion, but the recent format changes look nice with the base theme, but the dark theme is not very nice.

Current Dark Theme
image

PR Dark Theme
image

I'm not attached to the colors, but I just wanted to maybe start the conversation about how to handle common themes.

@bmaltais
Copy link
Owner

Thank you for this PR. Yeah... I did not think about the dark theme... You are right... I am not sure how we address theming in the application at the moment. I will merge your PR to fix the awful color scheme for people using a dark theme... and we can start to discuss theming support.

I just started to add better colors and some shadow effects to make navigating the options a bit more easy... I am sure there is much more improvements that could be made... but this was a test to see if users would like it... or hate it ;-)

@bmaltais bmaltais merged commit d8b5723 into bmaltais:sd3-flux.1 Sep 13, 2024
@wcole3 wcole3 deleted the dark-mode-styles branch September 13, 2024 23:24
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.

2 participants