-
Notifications
You must be signed in to change notification settings - Fork 14
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 #132
base: main
Are you sure you want to change the base?
Dark theme #132
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding a dark theme for people who prefer that is very welcome, thank you!
Is there are particular reason you didn't make table and input fields use light-on-dark text?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The style looks good to me now, just some space issues in the CSS left: one double space, and a bunch of lines with space at EOL (unfortunately the GitHub web diff doesn't highlight that, a local git diff
should, and any editor that supports Editorconfig should prevent it). 😺
Just a quick note. I have a custom CSS overlay that I wrote that I apply to the extension's settings UI. It does a dark theme as well as makes some important (IMO) UX improvements. I'm super busy for another week, but if you ping me in about 4-5 days, I can share the code with you if you like. |
The configuration page is too bright. I've added some dark colors for people who prefer them.
To test this style: Change website appearance settings in Firefox.