Clearing default `gui=` settings.

[ndreas]

With the recent changes on how the gui= parameter is handled, it is no longer cleared if left empty. In addition, there is no way to set gui=NONE.

Neovim defines a number of default gui= settings, for example for hi TabLine, and I have had to add gui = "nocombine" to the rules that I want to clear.

[rktjmp]

Are you able to provide a small example of how you're doing things?

Are you mixing gui= and bold=true etc? That's not recommended but if you are we can discuss ergonomic improvements perhaps.

If I define my group TabLine { fg = red }, the underline is removed as expected, if I inherit from it I can set gui="" or gui="NONE" to clear any that I do intentionally set. If I never define TabLine I can't inherit from it so that seems like a non-factor.

Perhaps there are groups that naturally link to tabline, or I guess text inside a tabline?

Having trouble reproducing or understanding the issue.

[ndreas]

I wonder if it's due to the way I build my theme. I use TabLine { bg = dark_grey, fg = grey }, and then I build using a custom Shipwright file. The build uses require("shipwright/transform/lush").to_vimscript which to my eyes does not respect NONE here: https://github.com/rktjmp/lush.nvim/blob/main/lua/shipwright/transform/lush/to_vimscript.lua#L42

[ndreas]

Files are here: https://gitlab.com/ndreas/vim-config/-/tree/main/lua/ndreas-theme I don't use Lush dynamically, but rather build my theme into colors/.

[rktjmp]

Ah yes, the vimscript exporter is kind of "best effort", not surprised to see it have a rough edge.

Previously, if gui was not set, it would end up as gui=NONE in the viml

https://github.com/rktjmp/lush.nvim/blame/0517452a74fe25f6a62cacb75e4b25bb1dcb2803/lua/shipwright/transform/lush/to_vimscript.lua#L53-L55

Can you try the fix-91 branch, I think that will do what you want and restore the old behaviour.

[ndreas]

Yep it works great!

[rktjmp]

Fixed in 6923ccef17c5276bc5d3cca907082e19ee165d55

Thanks for reporting the issue!