search

overcache / NeoSolarized

NeoSolarized: A fixed solarized colorscheme for better truecolor support.
MIT License
517 stars 99 forks source link

IncSearch visibility #19

Open fgrsnau opened 5 years ago

fgrsnau commented 5 years ago

Thanks for creating this colorscheme! I am happily using it with NeoVim.

I am wondering if the visual difference between IncSearch and Search is intended. Search sets the background color and is very easy to spot. IncSearch only sets the foreground color and I am unable to spot the incremental search result on the screen as the difference is so subtle. This makes incremental search not very useful.

I am very satisfied with the following patch. Maybe you want to incorporate it or provide an additional config flag?

@@ -466,7 +466,7 @@ exe "hi! StatusLineNC"   .s:fmt_none   .s:fg_base00 .s:bg_base02 .s:fmt_revbb
 exe "hi! Visual"         .s:fmt_none   .s:fg_base01 .s:bg_base03 .s:fmt_revbb
 exe "hi! Directory"      .s:fmt_none   .s:fg_blue   .s:bg_none
 exe "hi! ErrorMsg"       .s:fmt_revr   .s:fg_red    .s:bg_none
-exe "hi! IncSearch"      .s:fmt_stnd   .s:fg_orange .s:bg_none
+exe "hi! IncSearch"      .s:fmt_revr   .s:fg_orange .s:bg_none
 exe "hi! Search"         .s:fmt_revr   .s:fg_yellow .s:bg_none
 exe "hi! MoreMsg"        .s:fmt_none   .s:fg_blue   .s:bg_none
 exe "hi! ModeMsg"        .s:fmt_none   .s:fg_blue   .s:bg_none