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

IncSearch visibility #19

Open
fgrsnau opened this issue Apr 5, 2019 · 0 comments
Open

IncSearch visibility #19

fgrsnau opened this issue Apr 5, 2019 · 0 comments

Comments

@fgrsnau
Copy link

fgrsnau commented Apr 5, 2019

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant