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

DiffAdd is grey #290

Closed
ambroisie opened this issue Oct 18, 2023 · 8 comments
Closed

DiffAdd is grey #290

ambroisie opened this issue Oct 18, 2023 · 8 comments

Comments

@ambroisie
Copy link

Describe the bug
After updating the plug-in, the color of DiffAdd changed from green to grey.

To Reproduce
Steps to reproduce the behavior:

  1. Use git commit with commit.verbose=true configured.
  2. Notice that the wrong color is being used.

Expected behavior
I would expect it to be green, as it used to be.

Screenshots
Screenshot
:highlight

Additional context
I'm not sure why it changed, as looking at the code/:highlight everything looks in order. I don't override any color in my config either.

@ambroisie
Copy link
Author

This is in the terminal with termguicolors enabled.

@ambroisie
Copy link
Author

This seems to be related to tmux, as running nvim outside of tmux shows the correct colors (though more muted than I rembered).

image
image

@ambroisie
Copy link
Author

Here is how it looked before:

In tmux:
image

Outside of tmux:
image

This is with tmux 3.3a in all screenshots.

@ambroisie
Copy link
Author

The change in look seems to be from d3b86c2 and related to #239. I will be looking into how to reverse the change, as I dislike the more muted look.

I will leave the bug open, as I think this might be an issue for other people, even if I work around it through going back to the previous style.

@mrvon
Copy link

mrvon commented Oct 20, 2023

Agree, Old style more confortable and clear to me, I use tag to revert it right now.

Plug 'ellisonleao/gruvbox.nvim', {'tag': '5e697fa640944dd4a98748d15c6e0a7725777551'}

@ambroisie
Copy link
Author

Alright, so I fixed the tmux/native terminal issue: turns out my tmux was not correctly configured for 24-bit colour!

See this SO answer.

I still dislike the new styling however, so thankfully #291 gets us most of the way there (not quite: I want both the old colours and the low-key high-lighting from reverse = true).

ambroisie added a commit to ambroisie/nix-config that referenced this issue Oct 26, 2023
I dislike the new style of diff [1].

Thankfully somebody wrote a patch to configure it [2] (though not
completely to my liking, so the local patch here is a bit different).
I used it as a basis, but made it always revert, rather than
configurable.

[1]: ellisonleao/gruvbox.nvim#290
[2]: ellisonleao/gruvbox.nvim#291
ambroisie added a commit to ambroisie/nix-config that referenced this issue Oct 26, 2023
I dislike the new style of diff [1].

Thankfully somebody wrote a patch to configure it [2] (though not
completely to my liking, so the local patch here is a bit different).
I used it as a basis, but made it always revert, rather than
configurable.

[1]: ellisonleao/gruvbox.nvim#290
[2]: ellisonleao/gruvbox.nvim#291
@ellisonleao
Copy link
Owner

hey guys, thanks for opening the issue. I've made a comment in the PR so i really think this should be changed using overrides in your config. LMK if that work for you guys

ambroisie added a commit to ambroisie/nix-config that referenced this issue Oct 26, 2023
I dislike the new style of diff [1].

After somebody wrote a patch [2] I finally started experimenting with
what looked best to me.

This is using the old vibrant colours, which I like better. And avoids
using `reverse = true` to not break high-lighting during visual
selection.

This is using an overlay as it is _much_ easier to refer to the internal
colours in a `dark`/`light` agnostic way that way instead of the
intended "use the palette way" (due to breaking changes in [3] which,
incidentally, is the MR which changed diff high-lighting).

[1]: ellisonleao/gruvbox.nvim#290
[2]: ellisonleao/gruvbox.nvim#291
[3]: ellisonleao/gruvbox.nvim#280
@ambroisie
Copy link
Author

@ellisonleao I opened a discussion about an issue I have with overrides: #321.

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

No branches or pull requests

3 participants