-
Notifications
You must be signed in to change notification settings - Fork 600
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 Theme #71
Comments
A better solution is to use the direct css selector prefers-color-scheme instead of cookies. Thank you. |
@jinxka thanks for pointing that out, I didn't know that existed! I think there are cases where a user may want to override that setting, but it'd definitely be good to allow the user's OS selected theme to be inherited. Like how Slack allows for the user to sync theme with OS settings or override for that specific application: |
@isaisabel Thanks for the implementation. I might have overseen it, but I am afraid I couldn't find the option to turn on the dark theme. In the help menu at the top right there's no documentation of the dark theme, could it please be added there? In short: How do I turn on the dark theme? Thanks! |
Hi @agg-mb, The dark theme control is available on the new tab interface here: You seem to be correct that the USAGE document does not mention the dark theme. I'm not actually maintaining ATT&CK Navigator anymore so I'll leave that potential change to the current maintainers (@clemiller from the look of it). |
Thanks so much @isaisabel ! One more question @clemiller : |
An accessibility nice-to-have would be multiple visual themes for the Navigator. People working in low-light environments may find the white background of our current theme blinding. Adding an alternate color scheme with a dark background and light text (a "dark theme") would improve usability for demos or workspaces with lower light levels.
Optimally the user would be able to change the theme live in the application (perhaps using a dropdown next to the help button in the top right). The application should remember the previous theme selection using cookies.
The text was updated successfully, but these errors were encountered: