altercation / vim-colors-solarized

precision colorscheme for the vim text editor
http://ethanschoonover.com/solarized
6.58k stars 1.76k forks source link

SignColumn highlighting could do with some lovin' #26

Open saj opened 13 years ago

saj commented 13 years ago

Solarized's default sign highlighting is a little harsh on the eyes: I see purple on white with background=dark.

Adding s:bg_base02 was good enough for me:

diff --git a/colors/solarized.vim b/colors/solarized.vim
index 70f5223..0c78d59 100644
--- a/colors/solarized.vim
+++ b/colors/solarized.vim
@@ -654,7 +654,7 @@ exe "hi! DiffDelete"     .s:fmt_none   .s:fg_red    .s:bg_base02
 exe "hi! DiffText"       .s:fmt_none   .s:fg_blue   .s:bg_base02 .s:sp_blue
     endif
 endif
-exe "hi! SignColumn"     .s:fmt_none   .s:fg_base0
+exe "hi! SignColumn"     .s:fmt_none   .s:fg_base0  .s:bg_base02
 exe "hi! Conceal"        .s:fmt_none   .s:fg_blue   .s:bg_none
 exe "hi! SpellBad"       .s:fmt_curl   .s:fg_none   .s:bg_none    .s:sp_red
 exe "hi! SpellCap"       .s:fmt_curl   .s:fg_none   .s:bg_none    .s:sp_violet

Thanks!

jwhitley commented 11 years ago

+1 This is particularly egregious when Solarized is used with @airblade's vim-gitgutter.

jayekub commented 10 years ago

+1

ZeroKnight commented 10 years ago

:+1:

adolfoabegg commented 10 years ago

+1

lkraav commented 10 years ago

+1 .s:bg_base02 works perfectly

csierra commented 10 years ago

+1 works great with vim-gitgutter

airblade commented 10 years ago

Works for me (vim-gitgutter author)!

mechanicles commented 10 years ago

:+1:

neiled commented 10 years ago

:+1:

jrhorn424 commented 10 years ago

:+1:

shuangwu commented 10 years ago

:+1:

flybayer commented 9 years ago

:+1:

vitalk commented 9 years ago

:+1:

jacob-keller commented 5 years ago

Definitely +1 for me as well.