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

Allow users to use dark mode #1116

Merged
merged 2 commits into from
Nov 6, 2023
Merged

Allow users to use dark mode #1116

merged 2 commits into from
Nov 6, 2023

Conversation

brylie
Copy link
Contributor

@brylie brylie commented Nov 6, 2023

Related to #1115

It is nice to allow users to choose light or dark mode based on their own needs, such as sensitivity to bright light.

It is nice to allow users to choose light or dark mode based on their own needs, such as sensitivity to bright light.
@ogabrielluiz
Copy link
Contributor

Hey @brylie
Thanks for the PR.

Could you check that all the pages in the docs work well in Dark mode, please? This is probably the reason we have that deactivated.

@brylie
Copy link
Contributor Author

brylie commented Nov 6, 2023

The dark mode works overall, but isn't 100% covered. For example there are some page elements that have light backgrounds (pictured below.) However, having most things in dark mode is a significant improvement to reducing eye strain even when a few items still remain with light background. So, I'd urge the team to not let perfect be the enemy of good on this PR and consider it as an incremental step towards proper dark mode support.

Code blocks and inline code use GitHub Light theme

image

image

@brylie
Copy link
Contributor Author

brylie commented Nov 6, 2023

Leaving the dark/light toggle enabled and switching to github-dark mode for the code and inline code panels seems like a good compromise.

image

image

Copy link
Contributor

@ogabrielluiz ogabrielluiz left a comment

Choose a reason for hiding this comment

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

LGTM

@ogabrielluiz ogabrielluiz merged commit 20a313c into langflow-ai:dev Nov 6, 2023
3 checks passed
@brylie brylie deleted the patch-1 branch November 6, 2023 17:18
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.

None yet

2 participants