-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Themes update #3587
Themes update #3587
Conversation
It's done @archseer Let me know if you'd want side by side screenshots for any of them :) |
Thank you! I'm ok with monokai and everforest changes |
Looking at it here it has sky for bg. That's wrong isn't it? Don't know what's happened, I'll fix it to shade later tonight. Thanks! Penumbra only set fg for some fields that expects a bg. Most important were |
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.
Snazzy looks good 👍
Co-authored-by: Ivan Tham <[email protected]>
Co-authored-by: Ivan Tham <[email protected]>
97bfa82
to
db31062
Compare
Rebased. |
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.
Looks good to me. I guess we don't really have to wait for all authors to get come back and tick this, those that wanted some changes can just submit a pull request separately later, but it's good to still notify them.
I'm not the author of the solarized themes by the way. |
* lint: snazzy * lint: pop-dark * lint: penumbra+ * lint: noctis_bordo * lint: ingrid * lint: flatwhite * lint: doom_acario_dark * lint: dark_plus * lint: boo_berry * fix: use 47345E per author instruction * lint: bogster * lint: everforest * lint: acme * lint: serika * fix: use nasty_red per author suggestion * lint: nord * lint: dracula * lint: gruvbox * fix: cursorline gruvbox * lint: autumn * fix: no fg for cursorline * lint: ayu * lint: rose_pine * lint: monokai * lint: solarized * lint: spacebones * fix: penumbra bg!=sky * Update runtime/themes/ayu_light.toml Co-authored-by: Ivan Tham <[email protected]> * Update runtime/themes/ayu_light.toml Co-authored-by: Ivan Tham <[email protected]> * fix: fg for ui.help thanks @pickfire * fix: ingrid with input from @intarga Co-authored-by: Ivan Tham <[email protected]>
* lint: snazzy * lint: pop-dark * lint: penumbra+ * lint: noctis_bordo * lint: ingrid * lint: flatwhite * lint: doom_acario_dark * lint: dark_plus * lint: boo_berry * fix: use 47345E per author instruction * lint: bogster * lint: everforest * lint: acme * lint: serika * fix: use nasty_red per author suggestion * lint: nord * lint: dracula * lint: gruvbox * fix: cursorline gruvbox * lint: autumn * fix: no fg for cursorline * lint: ayu * lint: rose_pine * lint: monokai * lint: solarized * lint: spacebones * fix: penumbra bg!=sky * Update runtime/themes/ayu_light.toml Co-authored-by: Ivan Tham <[email protected]> * Update runtime/themes/ayu_light.toml Co-authored-by: Ivan Tham <[email protected]> * fix: fg for ui.help thanks @pickfire * fix: ingrid with input from @intarga Co-authored-by: Ivan Tham <[email protected]>
* lint: snazzy * lint: pop-dark * lint: penumbra+ * lint: noctis_bordo * lint: ingrid * lint: flatwhite * lint: doom_acario_dark * lint: dark_plus * lint: boo_berry * fix: use 47345E per author instruction * lint: bogster * lint: everforest * lint: acme * lint: serika * fix: use nasty_red per author suggestion * lint: nord * lint: dracula * lint: gruvbox * fix: cursorline gruvbox * lint: autumn * fix: no fg for cursorline * lint: ayu * lint: rose_pine * lint: monokai * lint: solarized * lint: spacebones * fix: penumbra bg!=sky * Update runtime/themes/ayu_light.toml Co-authored-by: Ivan Tham <[email protected]> * Update runtime/themes/ayu_light.toml Co-authored-by: Ivan Tham <[email protected]> * fix: fg for ui.help thanks @pickfire * fix: ingrid with input from @intarga Co-authored-by: Ivan Tham <[email protected]>
This is a collection of changes made to force themes to handle in a predictable way. It also enforces a response to enabling features (like
cursorline
) so users who want these features will still see them even if they change themes.I've tried my best to stick to the spirit of the theme, but some themes relied so much on terminal colors that I just had to guess and base some assumptions from other editors with similar themes.
Status from original authors (in terms of approvals)