nordtheme / nord

An arctic, north-bluish color palette.
https://www.nordtheme.com
MIT License
6.1k stars 170 forks source link

LineNr highlight is too dark compared to comments #233

Closed ghost closed 1 year ago

ghost commented 2 years ago

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 commented 2 years ago

Hi @yiyang-dev :wave:, thanks for your contribution :+1:

The rationale behind nord3_bright is documented in https://github.com/arcticicestudio/nord/issues/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.