[BUG] Transparency not working on blink.cmp box

[helixoid]

Describe the bug

As I mentioned in #404, blink.cmp completion menu box is not transparent event though I set the tranparency to true.

Expected behaviour

It should work as expected.

Screenshots

[eeeXun]

You can change the winhighlight

require("blink.cmp").setup({
    completion = {
        menu = {
            winhighlight = "Normal:Normal,FloatBorder:Normal,CursorLine:Visual,Search:None",
        },
    },
})

[helixoid]

But how come catppuccin does it by default with transparency turned on??
I don't have to touch blink's config at all in catppuccin.

[eeeXun]

I guess they already mapped the menu background to Normal.

You can check these line https://github.com/ellisonleao/gruvbox.nvim/blob/main/lua/gruvbox.lua#L533-L564.

If you don't want to change blink's config. You can override these groups

[helixoid]

Well wouldn't it be better to add this to gruvbox colorscheme by default.
Many wouldn't know how to do this unless they read this bug report.
Also it' would be better to add an option for making floating window/boxes to be transparent or not like tranparent_floating = true,

If it's reasonably simple to enable by default, please do so, as I & many people exclusively use the Gruvbox colorscheme.

[helixoid]

Any updates on this one?
Even the gruvbox-material has it by default and it makes sense.
I know the winhighlight option works but making a default behavior would be great.

[eeeXun]

I'm not the author. But I don't think it will have any update here. Because this a personal thing.

Given the default menu highlight group is Pmenu. And you don't want to change the blink config. What you can do is change gruvbox config.

require("gruvbox").setup({
    overrides = {
        Pmenu = { link = "Normal" },
    },
})

[helixoid]

Yeah that does it.
Thanks.