-
Notifications
You must be signed in to change notification settings - Fork 21
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 mode switcher seems broken #129
Comments
Hi @christian-bromann thanks for the report. And the clicky gif! :) It's basically a 3-way toggle: force light mode ☀️, force dark mode 🌗, and "off" which uses system default -- in your case, the system theme is dark. so when you "disabled" the dark mode icon 🌙 it removes the local storage entry and the site uses your system's preferred theme. Any ideas how this might be communicated better? It has come up 3 times (once on our team, 2 from other readers) but I imagine others are confused and don't report. No idea how to measure that. One option is to remove the "off" mode altogether, but I personally like the control of setting something back to a sort of default. Maybe I should just get over it and we go with an interaction that's more common like found on Astro's docs: |
Ahh, yeah now that you mention it, it does make sense, however not on first sight. Maybe one solution could be:
Basically only store the mode if it differentiates from the system default. I think it could create the same UX without that confusion. Like there is no reason to store the dark mode preference if it is already the system default, at least I don't see a use case for it. |
I feel like this was an interesting idea but I have had multiple people tell me it is broken. |
It seems that the dark mode switcher is not displayed correctly. The sun and moon don't switch making the moon button useless, see:
The text was updated successfully, but these errors were encountered: