jdinhify / vscode-theme-gruvbox

Gruvbox theme for vscode based on https://github.com/morhetz/gruvbox
https://marketplace.visualstudio.com/items?itemName=jdinhlife.gruvbox
MIT License
274 stars 63 forks source link

Variant closer to vim's gruvbox theme #36

Closed prokolyvakis closed 2 years ago

prokolyvakis commented 3 years ago

First of all, thanks for your amazing work!

I would like to ask whether is possible to add a variant which is closer to the colour scheme of vim's gruvbox theme.

Thanks in advance!

jdinhify commented 3 years ago

Hi @prokolyvakis , Thanks for taking time to open this.

Could you please elaborate on the "closeness" to the vim colour scheme? Is this about the colour accuracy, or syntax highlighting, or anything else?

Maybe a bit of background - the starting intention of this theme was to match as much as I could from the original vim theme, but there were some factors that made it not possible (with my time and expertise), most important ones were:

So, in the end, I just tried to use the original vim theme as an aspirational reference, with the strict rule was to use only colours in the original palette.

Perhaps we could start with some improvement suggestions for your programming languages, then we'll try and see how we would introduce the changes? PRs are welcomed too 👍

My thought is that if the changes aren't that much different from the current theme, we could introduce them to the existing theme. But if the changes would make everything rendered differently, then perhaps another variant as suggested, or a fork of this theme & published under another extension (I'm happy for people to fork this project, modify & publish if they want).

prokolyvakis commented 3 years ago

Hello, thank you a lot for your reply. First of all, I have to apologize for the vague "closer to the colour scheme". Using the theme the last month, I have to admit that the colour scheme is amazingly close to the gruvbox and the user experience in VS code is amazing. So thank you for your hard work! :)

The major difference that I managed to notice is only the background colour which is basically, at least to my eyes closer to grey. I would love to help, but I have to admit that my knowledge of on this regard is rather restricted. If you want I can also upload an image of how vim looks on my computer to better illustrate the differences and get your feedback.

Thank you very much for your reply and your amazing work once again!

jdinhify commented 2 years ago

Hey @prokolyvakis , Apologies for the late reply 🙏

I have checked with vim and can confirm that the background colour in vim by default isn't displayed exactly as the colour specified by gruvbox. The reason is that vim seems to use terminal colours by default, and displaying colours slightly different from the colours provided by the theme.

When using vim with terminals like iTerm or Hyper, I had the gui color turned on, which actually makes vim display the correct background colour ("correct" as in I verified by using a colour checker to check against the values in provided palette from https://github.com/morhetz/gruvbox)

set background=dark
set termguicolors
colorscheme gruvbox

screenshots

with-termguicolors without-termguicolors

I'm not sure if this is the issue that you're seeing?

prokolyvakis commented 2 years ago

Hello, I truly apologize for the late response but somehow I did not see the notification of your reply. This is exactly what I was referring to, but I was not able to understand which was the factor that led to different colours!

jdinhify commented 2 years ago

Thanks for confirming @prokolyvakis

I think turning off vim's guicolors makes the palette becomes a totally different thing, thus we won't be making change to this theme as it's referring to the true colours specified in gruvbox's colour palette. I'll close this issue for now, but let me know if you have further advices/thoughts.