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

Contrast between hl-DiffChange and hl-DiffText on the light theme is too low... right? #57

Open
aintyourcat opened this issue Mar 25, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@aintyourcat
Copy link

aintyourcat commented Mar 25, 2023

Hello! thanks for the colorscheme, been using it for a while now :D

However, as the title said, i have a little problem regarding the colors of the diff on the light theme, to be more specific with the changed part (did a little research, i think its related with the DiffChange and DiffText highlights, but I'm unsure about it).

Here is how it looks like when i did :Gvdiffsplit (with fugitive.vim)...

2023-03-25_15-47

... compared with vscode.nvim.

2023-03-25_15-48

@savq
Copy link
Owner

savq commented Apr 4, 2023

Yes, this could be improved! The saturation could be increated to have more contrast.

It'd also be a good idea to use a different hue for DiffText. In dark mode it's not that bad, but in light mode it's really hard to distinguish the light blue from the light purple (because of the Abney effect).

I'm not sure what would be better. The red in your screenshot has good contrast, but red is usually used for deleted text 🤔 Maybe keep magenta for DiffChange and use red just for DiffText?

@savq savq added the enhancement New feature or request label Jun 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants