-
Notifications
You must be signed in to change notification settings - Fork 151
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
Add dark mode compatibility to Twinkle #2023
base: master
Are you sure you want to change the base?
Conversation
P.S. If you make future pull requests, you should probably do it on a different branch in your repo than master. Can get buggy especially if you do multiple pull requests at the same time. |
@@ -304,3 +349,39 @@ th.morebits-mbox-text, td.morebits-mbox-text { | |||
padding: 0.25em 0.9em; | |||
width: 100%; | |||
} | |||
|
|||
/* Twinkle config */ |
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.
Twinkle-specific styles should go to twinkle.css, not here.
color: var(--color-subtle, #54595d); | ||
} | ||
|
||
/* Override select2 silliness */ |
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.
--morebits-color-info: #6BDB6B; | ||
--morebits-color-warning: #FFC7B3; | ||
--morebits-color-titlebar-links: #8BADDF; | ||
--morebits-bgcolor-dialog: #17244A; |
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.
This is subjective, but I think the blue on black looks terrible. Might want to use a shade of grey instead.
I have tested locally, it seems to work.