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

Update dark mode guidance #429

Closed
wants to merge 1 commit into from
Closed

Conversation

jannuary
Copy link
Contributor

It has been obsolete since libadwaita anyway, but it is even more obsolete now with color scheme preference.

It has been obsolete since libadwaita anyway, but it is even more obsolete now with color scheme preference.
@xou816
Copy link
Owner

xou816 commented Jan 14, 2022

I have not had the chance to try out just yet! Curious to see how it works. If the system preference is set to dark or light, what does our PreferLight or PreferDark setting do exactly? Are we right to use these values?

@jannuary
Copy link
Contributor Author

If the system preference is set to dark or light, what does our PreferLight or PreferDark setting do exactly?

System preference has three options: "Light" (not exposed in elementary or GNOME afaik), "Default" and "Dark".

With prefers-dark-mode on, "Default" or "Dark" = dark, "Light" = light;
with it off, "Light" and "Default" = light, "Dark" = dark.

Also see Alexander's blog post

Having PreferLight/PreferDark isn't ideal, but when we have an actual preferences window, we could have a System/Light/Dark tri-state toggle that maps to PreferLight/ForceLight/ForceDark

@xou816
Copy link
Owner

xou816 commented Jan 14, 2022

We could make it a tristate thing already, or just have Default (ie system preference) vs PrefersDark in the meantime!

@jannuary
Copy link
Contributor Author

It could be yeah, though that also wouldn't make sense on systems without color scheme prefence.

Also, there is no "Default" from our side, just Prefer/Force Light or Dark

@Diegovsky Diegovsky closed this Jul 14, 2024
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

3 participants