Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: gitsigns background is not transparent when disable_background is set #186

Closed
jamestansx opened this issue Dec 17, 2023 · 1 comment · Fixed by #185
Closed

bug: gitsigns background is not transparent when disable_background is set #186

jamestansx opened this issue Dec 17, 2023 · 1 comment · Fixed by #185
Labels

Comments

@jamestansx
Copy link

Neovim version (nvim -v)

NVIM v0.9.4

Terminal

alacritty

Describe the bug

Gitsigns related background doesn't consistent with other sign column symbols as the bg is set to groups.background.

['GitSignsAdd'] = { fg = groups.git_add, bg = groups.background },
['GitSignsChange'] = { fg = groups.git_change, bg = groups.background },
['GitSignsDelete'] = { fg = groups.git_delete, bg = groups.background },

It will be nice to make it consistent with signcolumn background.

Steps To Reproduce

None

Expected Behavior

Gitsigns background should be similar to Signcolumn background

Repro

vim.o.packpath = "/tmp/nvim/site"

local plugins = {
  rose_pine = "https://github.com/rose-pine/neovim",
  -- ADD OTHER PLUGINS _NECESSARY_ TO REPRODUCE THE ISSUE
}

for name, url in pairs(plugins) do
  local install_path = "/tmp/nvim/site/pack/test/start/" .. name
  if vim.fn.isdirectory(install_path) == 0 then
    vim.fn.system({ "git", "clone", "--depth=1", url, install_path })
  end
end

require("rose-pine").setup({
  -- ADD ROSÉ PINE CONFIG THAT IS _NECESSARY_ FOR REPRODUCING THE ISSUE
})

vim.cmd("colorscheme rose-pine")
@jamestansx jamestansx added the bug label Dec 17, 2023
@mvllow
Copy link
Member

mvllow commented Dec 23, 2023

This should be fixed in the canary branch, thank you for reporting.

You could also add this to your config if you'd rather not try canary:

require("rose-pine").setup({
  highlight_groups = {
    GitSignsAdd = { bg = "NONE" },
    GitSignsChange = { bg = "NONE" },
    GitSignsDelete = { bg = "NONE" },
  }
})

@mvllow mvllow linked a pull request Dec 27, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants