Padding is now using gutter's color

[kchibisov]

Summary

After e8214fb the padding is now colored, which looks visually awful, since it's not really a padding anymore given that it closely visually attaches to line number.

An example of how it looks.

The solution would be to use either a background color or provide a distinct color for it(using background color likely makes the most sense).

Reproduction Steps

No response

Helix log

No response

Platform

Linux

Terminal Emulator

alacritty

Helix Version

helix 22.05 (791bf7e)

[archseer]

@pickfire

[pickfire]

@kchibisov Thanks for reporting this. Which theme are you using? I tried both onedark and default theme, it doesn't seem to have this issue.

[kchibisov]

Try base16-default-dark. But you can go through the themes(they live update) and you'll find themes that look differently with and without your patch I ve mentined.

[archseer]

Let's do this: If gutters.is_empty() then automatically apply a 1 char margin and remove the spacer after the line number. That way the space between the view and the code isn't a gutter component but instead a margin. A spacer can then still be used to apply space inside the gutter, between different components.