gruvbox-community / gruvbox

Retro groove color scheme for Vim - community maintained edition
MIT License
801 stars 62 forks source link

Markdown heading colors changed in #194

Closed josephmidura closed 9 months ago

josephmidura commented 9 months ago

The update from September caused heading colors in markdown files (.md) to change. Was this intentional? The orange color in the pre September version provided some welcome contrast compared to the color of the non-heading text.

Pre September 2023 colors: Screen Shot 2023-11-15 at 12 31 05 PM Post September 2023 colors:

Screenshot 2023-11-15 at 12 42 08 PM
cassanof commented 9 months ago

yeah not sure what happened in september... i will fork the old one; all my colorschemes are messed up

cassanof commented 9 months ago

https://github.com/cassanof/gruvbox

rbong commented 9 months ago

The update from September caused heading colors in markdown files (.md) to change. Was this intentional?

This appears to be because we now define new Neovim highlight groups - see #188 . This should now be fixed.

all my colorschemes are messed up

This is related to a degree, but has other factors as well.

In September, an effort was made to make language syntax colors much more consistent and customizable than they were before.

Languages that are part of Gruvbox are, for the most part, user submitted, and this means that basically all of them use different colors for what is essentially the same syntax. The definition of new Neovim highlight groups, particularly LSP highlight groups, means that consistency is a bigger problem than it ever was before.

That's why, with the new highlight groups, I also changed all existing highlight groups to use colors more consistently since this is a breaking change anyways and comes with a new major version.

I realize that if you have a ton of custom colorschemes, this will break those, and I apologize. Please realize that better consistency comes with advantages - for example HTML, XHTML, and JSX will all look the same - TOML and YAML will look the same - JS and JSON will look similar - C like languages will all look similar - etc.

I'd argue that it's much better to stick around and help us improve these new highlight groups - I'm also looking to implement a few options that would make things act the exact same as they were before, namely:

rbong commented 9 months ago

You can now use these settings if you want to change syntax groups to what they were before:

let g:gruvbox_legacy_language_groups = 1
let g:gruvbox_treesitter = 0

Please note that if you do use Treesitter for highlighting it still won't look right.

rbong commented 9 months ago

Please note that if you do use Treesitter for highlighting it still won't look right.

And it looks like from the screenshot in the OP that you do use Treesitter, which is what causes this issue.

josephmidura commented 9 months ago

Please note that if you do use Treesitter for highlighting it still won't look right.

And it looks like from the screenshot in the OP that you do use Treesitter, which is what causes this issue.

The screenshots in the OP were made in Terminal.app in MacOS 13.6.1 (Ventura). I don't user Treesitter. I can verify though, that setting this option sets the markdown syntax highlighting as it used to be pre-Sept. 2023:

let g:gruvbox_legacy_language_groups = 1

Thanks for the solution. I agree that increased standardization will be beneficial.

rbong commented 9 months ago

I don't user Treesitter

Interesting, I can't recreate it without Treesitter. I'm glad the solution works for you.