diff options
author | Evgeni Chasnovski <evgeni.chasnovski@gmail.com> | 2024-01-14 02:35:37 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-01-13 16:35:37 -0800 |
commit | 0c850add3ecedec5b7dea62f0dd152f3421b66fc (patch) | |
tree | 6f4968bbe6307d86e3d9408d66a8cf16b740e008 /test/functional/ui/embed_spec.lua | |
parent | bab5e7fe17db67aa24115eeb44fc68905b107d9e (diff) | |
download | rneovim-0c850add3ecedec5b7dea62f0dd152f3421b66fc.tar.gz rneovim-0c850add3ecedec5b7dea62f0dd152f3421b66fc.tar.bz2 rneovim-0c850add3ecedec5b7dea62f0dd152f3421b66fc.zip |
fix(colorscheme): default statusline groups usability #26921
Problem: Current values of `StatusLine` and `StatusLineNC` are currently
designed to be visually distinctive while being not intrusive.
However, the compromise was more shifted towards "not intrusive".
After the feedback, statusline highlight groups should be designed to:
- Make current window clearly noticeable. Meaning `StatusLine` and
`StatusLineNC` should obviously differ.
- Make non-current windows clearly separable. Meaning `StatusLineNC`
and `Normal`/`NormalNC` should obviously differ.
Solution:
- Update `StatusLineNC` to have more visible background.
- Update `StatusLine` to be inverted variant of `StatusLineNC`.
- Update `WinBar` and `WinBarNC` to not link to `StatusLine` and
`StatusLineNC` because it makes two goals harder to achieve.
- Update `TabLine` to link to `StatusLineNC` instead of `StatusLine`
to not be very visually intrusive.
Diffstat (limited to 'test/functional/ui/embed_spec.lua')
-rw-r--r-- | test/functional/ui/embed_spec.lua | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/functional/ui/embed_spec.lua b/test/functional/ui/embed_spec.lua index 9e08f7748e..e655ee1b54 100644 --- a/test/functional/ui/embed_spec.lua +++ b/test/functional/ui/embed_spec.lua @@ -26,7 +26,7 @@ local function test_embed(ext_linegrid) [3] = { bold = true, foreground = Screen.colors.Blue1 }, [4] = { bold = true, foreground = Screen.colors.Green }, [5] = { bold = true, reverse = true }, - [6] = { foreground = Screen.colors.NvimDarkGrey3, background = Screen.colors.NvimLightGrey1 }, + [6] = { foreground = Screen.colors.NvimLightGrey3, background = Screen.colors.NvimDarkGrey3 }, [7] = { foreground = Screen.colors.NvimDarkRed }, [8] = { foreground = Screen.colors.NvimDarkCyan }, }) |