-
Notifications
You must be signed in to change notification settings - Fork 33
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
Vim 0.10.0 breaks dim
colorscheme
#12
Comments
The affected highlights appear to be :highlight Function ctermfg=6
:highlight String ctermfg=1 I'm also working on a Lua rewrite of Dim to keep up with the more rolling nature of Neovim. |
I'd accept a patch for this. |
@schrobingus please make sure to ping back here if you do update it! much appreciated |
diff --git a/colors/default-light.vim b/colors/default-light.vim
index d49cff2..aafee36 100644
--- a/colors/default-light.vim
+++ b/colors/default-light.vim
@@ -24,6 +24,8 @@ highlight CursorLine cterm=underline
highlight MatchParen ctermbg=14
highlight Constant ctermfg=1
highlight Special ctermfg=5
+highlight Function ctermfg=6
+highlight String ctermfg=1
highlight Identifier cterm=NONE ctermfg=6
highlight Statement ctermfg=3
highlight PreProc ctermfg=5
diff --git a/colors/grim.vim b/colors/grim.vim
index 0853e91..eac0e88 100644
--- a/colors/grim.vim
+++ b/colors/grim.vim
@@ -5,6 +5,8 @@ let colors_name = "grim"
if &background == "light"
highlight Constant ctermfg=8
highlight Identifier ctermfg=0
+ highlight Function ctermfg=0
+ highlight String ctermfg=0
highlight PreProc ctermfg=0 cterm=bold
highlight Special ctermfg=0
highlight Statement ctermfg=0 cterm=bold
@@ -14,6 +16,8 @@ if &background == "light"
else
highlight Constant ctermfg=7
highlight Identifier ctermfg=15
+ highlight Function ctermfg=15
+ highlight String ctermfg=15
highlight PreProc ctermfg=15 cterm=bold
highlight Special ctermfg=15
highlight Statement ctermfg=15 cterm=bold @jeffkreeftmeijer Here is a patch applying the highlights. I also added this to Grim since it is also affected here, albeit I wasn't sure what highlight to substitute in, so I just did 0 and 15 respectively. |
I should get rid of this fork. Looks like stuff might be happening over on jeffkreeftmeijer#12 (comment)? I'm subscribed to the relevant issue.
|
There are still many colors broken since the 0.10.0 upgrade, #13 is still not enough. I ended up switching to another scheme so I won't be adding any more to #13. |
Seems like quite a few things have broken in the upgrade to neovim
0.10.0
. Particularly, string colors are no longer red.The text was updated successfully, but these errors were encountered: