Add more contrast for `document_highlight()`
Closed this issue · 13 comments
If anything, that should be user configurable as I personally see this theme's lower contrast as being a really good thing.
I think these could be a bit lighter on dark bg. On light bg they do seem to be more visible.
I may think about configuration here, but I am currently not convinced that it's worthwhile. Let's see when this change is applied.
I am currently refactoring and changing how shades are applied everywhere, so, as part of that, I will try to make these a bit lighter while keeping the low contrast vibe.
One of the difficulties here is to not overlap with the Search
highlight group.
In the screenshot below, dict
is highlighted as the result of a current search (the background is slightly lighter), whereas msg
is highlighted by vim.lsp.buf.document_highlight()
:
Using a colorful background for searches would solve that and "free" bg4
for the LspReferenceText
highlight group. Here is how Zenbones does it, for comparison:
By the way, out of interest I compared the APCA Lc1 value for LspReferenceText
in both dark and light modes. @strash is right, the contrast is objectively lower in dark mode (9 vs. 0).
Here are the values:
- bg_contrast = 'soft'
- bg_contrast = 'medium'
- bg_contrast = 'hard'
(edit: changed URLs to the non-productive "SAPC" version of the tool, which is more precise in very low contrast ranges.)
Similarly, the contrast for Search
is 12.9 vs. 0.
APCA is the algorithm used by W3C in the draft version of the WCAG accessibility guidelines. It is a very useful tool for designing colorschemes and ensuring that perceived contrasts are proportional between dark and light modes.
Footnotes
-
Lightness contrast ↩
Interesting. My eyes don't agree with this algorithm though 😀 Like if I try to make dark bg contrast match the contrast of light bg, it doesn't look like the same contrast at all...
I guess it also depends a lot on the environment and the display, e.g. I get much higher contrast on my laptop screen than on my monitor. And normally you would (at least I would) use light bg in bright environment compared to dark bg in darker environment. So even if light bg objectively does have more contrast, it would make sense when considering the practicality of described scenario.
My eyes don't agree with this algorithm [...]
even if light bg objectively does have more contrast, it would make sense when considering the practicality of described scenario.
That's fair, and to be clear I don't disagree with the existing highlights, and use primarily dark mode.
Keep in mind that although the algorithm takes into accounts the different needs of light and dark modes, it is geared towards accessibility use cases, primarily for text-on-background ratios (as opposed to background-on-background).
@strash would you say this is enough of a difference? I'm hesitant to add more contrast, as it will probably start being distracting. I would say it's still lower contrast than with light bg, but as I said before, dark bg is more targeted towards not-that-bright environments.
@ramojus It's hard to say. I see the difference between "before" and "after" but I also see a big difference between your screenshots and mine, so I'm a bit confused. But overall it's better now.
![Group 2242](https://private-user-images.githubusercontent.com/14233263/338006421-3b61e6d9-7772-4e40-80ab-df46f211c4f9.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQ4MTM1ODUsIm5iZiI6MTcyNDgxMzI4NSwicGF0aCI6Ii8xNDIzMzI2My8zMzgwMDY0MjEtM2I2MWU2ZDktNzc3Mi00ZTQwLTgwYWItZGY0NmYyMTFjNGY5LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MjglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODI4VDAyNDgwNVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTBlMmNjN2I3NDE5YTk1ZTU3ZDEyZDg1NzgwZjNjNzY3MjZjYzcyOTgwZTM0NDQyYThlYjQzMDQ4MDIxZTJjOWUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.yVPN7Dw72574yu3K83-fWN58CIEIWab8EeB-Dhzfadg)
@strash Probably not what you meant, but I'll say this just in case - I wasn't trying to match your suggestion (on your second screenshot) exactly. Instead, I was aiming for a more subtle change.
I think the difference between my before screenshot and your unmodified screenshot is that the font you are using has thicker strokes compared to the font that I'm using. That makes all of the colors appear brighter (as they have more area to fully show themselves, without antialiasing "weakening" them), and thus the contrast of bg layers is harder to see. I've checked with a color picker, and LspReferenceText
bg is the same color in both screenshots.
I think different fonts affecting the perceived contrast of bg layers may be enough of a reason for a config option. I'll think about it.
Indeed, it's very subtle change (0 0 16 -> 0 0 17 HSB 😀), but I think I see it better now. Thank you!
Better to keep it open for now. I'll add configuration for bg layers contrast and that's a bit related.
@strash I've added the ability to override shades in #45, but it's a breaking change, so it was merged to v1 branch. After longer usage, I ended not liking the changes in #40, so I will revert them. This is how you can replicate them (from v1 branch):
require 'mellifluous'.setup({
color_overrides = {
dark = {
colors = function(colors)
return {
bg2 = colors.bg:lightened(4),
bg3 = colors.bg:lightened(7),
bg4 = colors.bg:lightened(10),
bg5 = colors.bg:lightened(13),
}
end,
}
},
})
LspReferenceText
is linked to bg3 -- you can change it if you want even more contrast.
After a couple of weeks, v1 branch should be merged to main (when v1.0.0 will be released).