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

Vim 0.10.0 breaks dim colorscheme #12

Open
hexcowboy opened this issue Aug 14, 2024 · 6 comments · May be fixed by #13
Open

Vim 0.10.0 breaks dim colorscheme #12

hexcowboy opened this issue Aug 14, 2024 · 6 comments · May be fixed by #13

Comments

@hexcowboy
Copy link

Seems like quite a few things have broken in the upgrade to neovim 0.10.0. Particularly, string colors are no longer red.

@schrobingus
Copy link

schrobingus commented Aug 21, 2024

The affected highlights appear to be Function and String, so rebinding both of those should fix the issue altogether.

: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.

@jeffkreeftmeijer
Copy link
Owner

I'd accept a patch for this.

@hexcowboy
Copy link
Author

@schrobingus please make sure to ping back here if you do update it! much appreciated

@schrobingus
Copy link

schrobingus commented Aug 27, 2024

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.

@hexcowboy hexcowboy linked a pull request Aug 30, 2024 that will close this issue
jfly added a commit to jfly/vim-dim that referenced this issue Sep 18, 2024
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.
@clach04
Copy link

clach04 commented Sep 21, 2024

I'd accept a patch for this.

#13

@hexcowboy
Copy link
Author

hexcowboy commented Sep 21, 2024

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.

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

Successfully merging a pull request may close this issue.

4 participants