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

Use custom as dark mode #1023

Closed

Conversation

isaksamsten
Copy link

Hi!

Yet another PR from me :-).

Again, this can be stacked with #1021 and #1022.

The change introduces a configuration option which I've called use_custom_as_dark which treats toggle_dark_mode as toggle_custom_color. It plays really nicely with #1022 to allow the operating system to toggle light/dark mode but the user can configure the colors of the dark mode.

- remove unused code
- correctly indent the code with 4 spaces.
Automatically toggle between dark and light mode depending on the
desired mode from the operating system.

Introduces a new config option `use_system_theme` which if set
automatically set light or dark mode when starting depending on if the
current operating system theme is in light or dark mode. Then, if the
operating system mode changes, the change is reflected in Sioyek.
A new configuration option `use_custom_as_dark` which if set uses the
custom color mode as dark mode.
@ahrm
Copy link
Owner

ahrm commented Mar 26, 2024

Thanks, however, I think modifying the behavior of dark mode is a little confusing, so instead I added a use_custom_color_as_dark_system_theme option in 42c8ac9.

@ahrm ahrm closed this Mar 26, 2024
@isaksamsten
Copy link
Author

That's a good solution 👍

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