fix(highlight): omit DiffsClear bg on transparent themes
Problem: #178 set `DiffsClear.bg` to a dark fallback (`#1a1a1a`) on transparent themes, painting a visible solid rectangle where the terminal background should show through. Solution: only set `DiffsClear.bg` when `Normal.bg` is defined. Body lines are already protected by `line_hl_group` (`DiffsAdd`/ `DiffsDelete`), which stacks above `hl_group` bg regardless of priority.
This commit is contained in:
parent
59714381f1
commit
fca354c147
2 changed files with 18 additions and 3 deletions
|
|
@ -528,7 +528,11 @@ local function compute_highlight_groups(is_default)
|
|||
local blended_add_text = blend_color(add_bg, bg, text_alpha)
|
||||
local blended_del_text = blend_color(del_bg, bg, text_alpha)
|
||||
|
||||
vim.api.nvim_set_hl(0, 'DiffsClear', { default = dflt, fg = normal_fg, bg = bg })
|
||||
local clear_hl = { default = dflt, fg = normal_fg }
|
||||
if not transparent then
|
||||
clear_hl.bg = bg
|
||||
end
|
||||
vim.api.nvim_set_hl(0, 'DiffsClear', clear_hl)
|
||||
vim.api.nvim_set_hl(0, 'DiffsAdd', { default = dflt, bg = blended_add })
|
||||
vim.api.nvim_set_hl(0, 'DiffsDelete', { default = dflt, bg = blended_del })
|
||||
vim.api.nvim_set_hl(0, 'DiffsAddNr', { default = dflt, fg = add_fg, bg = blended_add })
|
||||
|
|
|
|||
|
|
@ -554,7 +554,7 @@ describe('diffs', function()
|
|||
diffs._test.set_hl_retry_pending(false)
|
||||
end)
|
||||
|
||||
it('uses dark fallback bg for DiffsClear when Normal.bg is nil (transparent)', function()
|
||||
it('omits DiffsClear.bg when Normal.bg is nil (transparent)', function()
|
||||
vim.api.nvim_get_hl = function(ns, opts)
|
||||
if opts.name == 'Normal' then
|
||||
return { fg = 0xc0c0c0 }
|
||||
|
|
@ -562,11 +562,22 @@ describe('diffs', function()
|
|||
return saved_get_hl(ns, opts)
|
||||
end
|
||||
diffs._test.compute_highlight_groups()
|
||||
assert.are.equal(0x1a1a1a, set_calls.DiffsClear.bg)
|
||||
assert.is_nil(set_calls.DiffsClear.bg)
|
||||
assert.is_table(set_calls.DiffsAdd)
|
||||
assert.is_table(set_calls.DiffsDelete)
|
||||
end)
|
||||
|
||||
it('sets DiffsClear.bg to Normal.bg on opaque themes', function()
|
||||
vim.api.nvim_get_hl = function(ns, opts)
|
||||
if opts.name == 'Normal' then
|
||||
return { fg = 0xebdbb2, bg = 0x282828 }
|
||||
end
|
||||
return saved_get_hl(ns, opts)
|
||||
end
|
||||
diffs._test.compute_highlight_groups()
|
||||
assert.are.equal(0x282828, set_calls.DiffsClear.bg)
|
||||
end)
|
||||
|
||||
it('blend_alpha controls DiffsAdd.bg intensity', function()
|
||||
local saved_config_alpha = diffs._test.get_config().highlights.blend_alpha
|
||||
diffs._test.get_config().highlights.blend_alpha = 0.3
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue