themes
themes copied to clipboard
tweak(base): re-balance symbol-overlay colors to improve visibility
Minor tweaks to colors so they're all more balanced, and a little bit more visible across different themes. Some colors are still slightly less visible on some themes compared to others, which I see no way around without having per-theme specific colors.
- [x] I searched the issue tracker and this hasn't been PRed before.
- [x] My changes are not on the do-not-PR list for this project.
- [x] My commits conform to the git conventions.
- [x] My changes are visual; I've included before and after screenshots.
- [x] Any relevant issues or PRs have been linked to.
After @seagle0128's comment I took a closer look at how the different colors I had picked in #750 appear on a long list of different themes.
The result is the subtle tweaks to color blending values in this PR, which I feel are overall a reasonable compromise to look good in most themes. I would not call it perfect in every theme, but more than good enough in my personal opinion. I'd be happy to adjust further if people disagree with my opinion though :)
I might have gone a little overboard, but below are a series of before/after screenshots for how this PR affects the various symbol-overlay-face-*
faces across a bunch of the themes.
Screenshots
doom-one
Before:
After:
doom-one-light
Before:
After:
doom-vibrant
Before:
After:
doom-dark+
Before:
After:
doom-molokai
Before:
After:
doom-monokai-classic
Before:
After:
doom-monokai-pro
Before:
After:
doom-nord
Before:
After:
doom-nord-aurora
Before:
After:
doom-nord-light
Before:
After:
doom-nova
Before:
After:
doom-solarized-dark
Before:
After:
doom-solarized-light
Before:
After:
doom-solarized-dark-high-contrast
Before:
After:
doom-tomorrow-night
Before:
After:
doom-tomorrow-day
Before:
After:
doom-zenburn
Before:
After:
doom-dracula
Before:
After:
doom-Iosvkem
Before:
After:
doom-opera
Before:
After:
doom-opera-light
Before:
After:
``
Before:
After:
@jimeh Thank you for the tweaking! I think you should also check the highlighting other words of different foreground colors.
@seagle0128 Ah, yeah good shout. I knew I should have finished my coffee before doing this 😅
@jimeh Any update for this PR? And I still think the colors should be lighter. The foreground colors of the characters may be very close to the background colors. The default faces of symbol-overlay
are very clear and obvious. I think user experience is better.
Please the screenshots below.
![image](https://user-images.githubusercontent.com/140797/193662005-5ab2730e-595f-4f04-8906-ff52f964d01f.png)
![image](https://user-images.githubusercontent.com/140797/193662910-9dd3a8ce-a2f6-4224-902b-042f996771a4.png)
My tweaks:
![image](https://user-images.githubusercontent.com/140797/193678172-5e8ac143-9ae9-4613-b91e-e2945738048f.png)
@seagle0128 Sorry for the silence, I'm gonna try to get back to this in the next few of days.
I agree with you and prefer the tweak you've done to what's in here right now.
My plan is to try a few different variations of background/foreground color combos across a whole bunch of the themes, and grab screenshots of it all to post here for comparison :)
@jimeh Thanks! Please proceed as your plan...