Closed habamax closed 2 years ago
Every colorscheme has
;;; "Standard" diff colors
Include: _diff
At the end of template.
Feel free to suggest adjustments to bg colors.
Ah!
@romainl I didn't touch 2 colorschemes: blue and darkblue.
They both have quite nice colors (at least in 256c :) )
I can convert them too if you don't mind or we can leave them as is.
And darkblue is actually also converted :) sorry.
We may have a problem with bold Statement:
I don't have it...
OK, it's a terminal emulator setting.
soften colors a bit:
darkblue
And I am still not sure about blue as of now:
Might be
PS, although it also looks good...
Anyway, I don't mind any other color selection ...
My comments…
235/#262626 is not distinct enough from black for the difference to matter so I would simply use ctermbg=16 (256c) and guibg=#000000 (GUI).
DiffDelete and DiffAdd are way too close too each other in 256c/GUI.
"green" and "yellow" work relatively well for people with normal vision but they are basically indistinguishable for people with deuteranopia (the most common form of color blindness):
funnily enough, it's "blue" and "green" that are confusing in case of tritanopia (which is very rare):
I'm still not sure about how much attention we should give to color blindness since there are relatively inexpensive glasses that supposedly correct that issue. But the lack of contrast between DiffDelete and DiffAdd is bothering.
How about this?
That works very well for normal vision. Great!
Blue difftext clashes with visual of my own colorscheme :(, but that is ok.
@romainl I wanted to propose this kind of change too
Yeah, :qa!
was tiring.
FWIW, I just spent some time in master in 16c and the diffs in this branch are a hell of a lot better.
The only slight issue I have is the lack of contrast of black on blue for DiffChange.
I tried with white on blue and sure enough the contrast is stronger, but now it sort of doesn't fit :-(:
@habamax what do you think?
we can have white fg on 16c and black for the rest
The problem is indeed worse on white bg. That's a good idea.
ctermbg=14
On the other hand black on blue is still readable for me on default xterm16
nice, what about white for diffadd?
btw, we could have a single _diff
with 2 definitions depending on background
Maybe. I tried it but colortemplate reported a bunch of errors so I opted for a different file.
I will fix it later, once we agree on colors
White fg is actually quite readable on dark bg too:
nice!
Closes #121