From c7cd8fc24c5047640e8d9cbd1c4221681427d8e5 Mon Sep 17 00:00:00 2001 From: Barrett Ruth <62671086+barrettruth@users.noreply.github.com> Date: Sun, 8 Mar 2026 16:08:36 -0400 Subject: [PATCH] fix(highlight): omit DiffsClear bg on transparent themes (#179) --- lua/diffs/init.lua | 6 +++++- spec/init_spec.lua | 15 +++++++++++++-- 2 files changed, 18 insertions(+), 3 deletions(-) diff --git a/lua/diffs/init.lua b/lua/diffs/init.lua index f2db40c..1f41725 100644 --- a/lua/diffs/init.lua +++ b/lua/diffs/init.lua @@ -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 }) diff --git a/spec/init_spec.lua b/spec/init_spec.lua index 91bbed1..5ee7532 100644 --- a/spec/init_spec.lua +++ b/spec/init_spec.lua @@ -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