Skip to content
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

tweak(base): re-balance symbol-overlay colors to improve visibility #754

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jimeh
Copy link
Contributor

@jimeh jimeh commented Sep 17, 2022

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.


  • I searched the issue tracker and this hasn't been PRed before.
  • My changes are not on the do-not-PR list for this project.
  • My commits conform to the git conventions.
  • My changes are visual; I've included before and after screenshots.
  • 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:

doom-one__before

After:

doom-one__after

doom-one-light

Before:

doom-one-light__before

After:

doom-one-light__after

doom-vibrant

Before:

doom-vibrant__before

After:

doom-vibrant__after

doom-dark+

Before:

doom-dark+__before

After:

doom-dark+__after

doom-molokai

Before:

doom-molokai__before

After:

doom-molokai__after

doom-monokai-classic

Before:

doom-monokai-classic__before

After:

doom-monokai-classic__after

doom-monokai-pro

Before:

doom-monokai-pro__before

After:

doom-monokai-pro__after

doom-nord

Before:

doom-nord__before

After:

doom-nord__after

doom-nord-aurora

Before:

doom-nord-aurora__before

After:

doom-nord-aurora__after

doom-nord-light

Before:

doom-nord-light__before

After:

doom-nord-light__after

doom-nova

Before:

doom-nova__before

After:

doom-nova__after

doom-solarized-dark

Before:

doom-solarized-dark__before

After:

doom-solarized-dark__after

doom-solarized-light

Before:

doom-solarized-light__before

After:

doom-solarized-light__after

doom-solarized-dark-high-contrast

Before:

doom-solarized-dark-high-contrast__before

After:

doom-solarized-dark-high-contrast__after

doom-tomorrow-night

Before:

doom-tomorrow-night__before

After:

doom-tomorrow-night__after

doom-tomorrow-day

Before:

doom-tomorrow-day__before

After:

doom-tomorrow-day__after

doom-zenburn

Before:

doom-zenburn__before

After:

doom-zenburn__after

doom-dracula

Before:

doom-dracula__before

After:

doom-dracula__after

doom-Iosvkem

Before:

doom-Iosvkem__before

After:

doom-Iosvkem__after

doom-opera

Before:

doom-opera__before

After:

doom-opera__after

doom-opera-light

Before:

doom-opera-light__before

After:

doom-opera-light__after

``

Before:

doom-xcode__before

After:

doom-xcode__after

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.
@seagle0128
Copy link
Contributor

@jimeh Thank you for the tweaking! I think you should also check the highlighting other words of different foreground colors.

@jimeh
Copy link
Contributor Author

jimeh commented Sep 17, 2022

@seagle0128 Ah, yeah good shout. I knew I should have finished my coffee before doing this 😅

@jimeh jimeh marked this pull request as draft September 17, 2022 16:17
@hlissner hlissner added is:faces Pertains to extending/improving support for Emacs or plugin faces theme:base Affects the base theme (i.e. all themes) labels Sep 17, 2022
@hlissner hlissner added this to the v2.4.0 milestone Sep 17, 2022
@seagle0128
Copy link
Contributor

seagle0128 commented Oct 3, 2022

@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

image

My tweaks:

image

@jimeh
Copy link
Contributor Author

jimeh commented Oct 4, 2022

@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 :)

@seagle0128
Copy link
Contributor

@jimeh Thanks! Please proceed as your plan...

@hlissner hlissner removed this from the v2.4.0 milestone Nov 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:faces Pertains to extending/improving support for Emacs or plugin faces theme:base Affects the base theme (i.e. all themes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants