Replies: 14 comments
-
I agree, and made a tool for helping out #3234. A lot of the issues you point out are not yet checked against, but they could :) |
Beta Was this translation helpful? Give feedback.
-
About base16_transparent, it is intentionally the same as the background. |
Beta Was this translation helpful? Give feedback.
-
Some of these might be intentional and fixes would be a bit opinionated. Pinging @two-six for Acme, Nord Light
Makes sense, added a note. |
Beta Was this translation helpful? Give feedback.
-
Thanks for the ping, I'll look into fixing acme. |
Beta Was this translation helpful? Give feedback.
-
It is very hard to read the undefined |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
Are the colors terminal-dependent? I have no problems in Konsole, can you tell me what terminal do you use? |
Beta Was this translation helpful? Give feedback.
-
It maybe/probably falls back to some base16 color set for the terminal? I use a gruvbox theme.
|
Beta Was this translation helpful? Give feedback.
-
Acme uses a mix of hard coded colors and colors derived from the terminal theme. That's why it can look pretty bad when a dark terminal theme is used. The issue for the Nord Light errors is that you used |
Beta Was this translation helpful? Give feedback.
-
Using Acme theme on iterm2 (acme theme also), and I got this white background for errors don't know if it is related to this issue. Thanks! EDIT: also the buffer line background. |
Beta Was this translation helpful? Give feedback.
-
@olbitset I have already fixed this in #5019, it just wasn't included in the latest release. You would need to compile helix from source or update |
Beta Was this translation helpful? Give feedback.
-
Thanks @two-six I included as a personal theme. What about the TOP bufferline (as the selection one), is it expected to have a gray background? If so what is the parameter in the toml config to adapt it to my need? |
Beta Was this translation helpful? Give feedback.
-
Do you mean foreground? I see background is light blue in your screenshot, not gray. Set I've never used bufferline before, but as you mentioned current configuration doesn't really fit this theme really well. I will open a pull request to change it from gray to black. |
Beta Was this translation helpful? Give feedback.
-
Sorry I was talking about the bg. Thanks for the inputs I will adapt to my preferences! |
Beta Was this translation helpful? Give feedback.
-
Some themes that come with helix need some polish.
acme: statusline, menu, maybe errors
base_16_transparent: menu bg same as standard bg (intentional, add border once it's possible)
nord_light: ui menu selected bg very similar to standard bg
rose_pine_dawn: ui menu selected bg same as standard bg
FIXED
dark_plus: indent guide bg
ayu_light: statusline fg, menu fg
Beta Was this translation helpful? Give feedback.
All reactions