-
Notifications
You must be signed in to change notification settings - Fork 252
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
Detect dark mode setting from browser/OS #96
Comments
👋 Hi Tsugi Team! Just a word of caution, we at AA found a usability issue with defaulting to dark mode. Users with eyesight issues immediately had concerns. The main feedback is users felt the desktop OS dark mode websites were different from mobile OS dark mode for apps. Instead, we are opting to have a switch in the user profile. Additional context |
Thanks - We will make sure to add an override in the profile once we make it work.
… On Dec 27, 2020, at 4:10 AM, Brett Wilcox ***@***.***> wrote:
👋 Hi Tsugi Team!
Just a word of caution, we at AA found a usability issue with defaulting to dark mode. Users with eyesight issues immediately had concerns.
Instead, we are opting to have a switch in the user profile.
Additional context
https://www.w3.org/WAI/fundamentals/accessibility-usability-inclusion/ <https://www.w3.org/WAI/fundamentals/accessibility-usability-inclusion/>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#96 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJEJJSITGEQZ7WBAQFQ3HTSW32Y3ANCNFSM4QR4ATEA>.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I believe some work was done here to support dark mode. The default should be the mode specified by the browser.
Most browsers supply this media query https://caniuse.com/#search=prefers-color-scheme
This can also be read in javascript. This article has the best practice ways to support this: https://web.dev/prefers-color-scheme/#finding-out-if-dark-mode-is-supported-by-the-browser
The text was updated successfully, but these errors were encountered: