Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

LineNr highlight is too dark compared to comments #233

Closed
ghost opened this issue Aug 16, 2022 · 1 comment
Closed

LineNr highlight is too dark compared to comments #233

ghost opened this issue Aug 16, 2022 · 1 comment

Comments

@ghost
Copy link

ghost commented Aug 16, 2022

Currently comments are set as such

call s:hi("Comment", s:nord3_gui_bright, "", s:nord3_term, "", s:italicize_comments, "")

However line numbers are set:

call s:hi("LineNr", s:nord3_gui, "NONE", s:nord3_term, "NONE", "", "")

s:nord3_term is actually quite difficult to see (as indicated in nordtheme/vim#227) where the solution was to :set termguicolors.

The same logic unfortunately seems to apply here too - the darker variant (term3) is used for line numbers making them really difficult to see even with termguicolors.

Would you accept a PR to change to:

call s:hi("LineNr", s:nord3_gui_bright, "NONE", s:nord3_term, "NONE", "", "")

EDIT: I suppose this isn't entirely consistent since we have other areas (folds, etc) which are also using nord3. I don't completely understand the rationale behind using nord3 vs nord3_bright.

I'm not sure what the solution here is. I can override this locally in my vimrc I suppose..

@svengreb
Copy link
Member

Hi @yiyang-dev 👋, thanks for your contribution 👍

The rationale behind nord3_bright is documented in #94 that also includes information why only comments are affected. When nord3 is really that hard to see I can also recommend to make sure that your system uses the correct color profile for your monitor or maybe use one that is explicitly optimized for text. I also use a sRGB color profile (IEC61966-2.1) on my macOS system for my Samsung wide-screen monitor to make sure that colors are "real" (color codes match the rendered color) for best experience for design tasks and also the smoothest and "crispest" rendering for text. nord3 is good as it is and I often have to make sure that users understand the fact that a good local setup matters.

Changing the highlighting of line numbers for all users will cause more problems as it solves, but you can still simply resolve it on your own by customizing the Nord styles to match your own preferences without having to change the source code at all.

@svengreb svengreb assigned ghost Aug 18, 2022
@svengreb svengreb unassigned ghost Jun 8, 2023
@svengreb svengreb transferred this issue from nordtheme/vim Jun 8, 2023
@nordtheme nordtheme locked and limited conversation to collaborators Jun 8, 2023
@svengreb svengreb converted this issue into discussion #234 Jun 8, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Projects
None yet
Development

No branches or pull requests

1 participant