melange-nvim icon indicating copy to clipboard operation
melange-nvim copied to clipboard

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

Open aintyourcat opened this issue 2 years ago • 1 comments

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

aintyourcat avatar Mar 25 '23 07:03 aintyourcat

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 avatar Apr 04 '23 22:04 savq