Use vim.api.nvim_set_hl()

[gegoune]

With neovim/neovim@4aa0cdd merged in it is possible now to use vim.api.nvim_set_hl(0, …) to set highlights for global :highlight namespace.

I have noticed in nordic's code that you are using vim.highlight.create but couldn't find it in documentation. What is it?

[andersevenrud]

This theme uses an internal method:

https://github.com/neovim/neovim/blob/5be2cdd913ce94bf3b88ce70042fbf7a92066459/runtime/lua/vim/highlight.lua#L6-L13

Suppose this ideally should be copied into this repo because it's marked as private, hence no guarantee to keep working in later releases :)

[gegoune]

So it uses vim.cmd, that makes sense. I think I could try to give PR implementing vim.api.nvim_set_hl() a go.

[andersevenrud]

To use the current (i.e. 0.6.1) nvim_set_hl signature you can just do this:

local ns = vim.api.nvim_create_namespace('nordic')
vim.api.nvim_set_hl(ns, ...)

Not really sure if it's worth supporting the 0 argument value until it hits a release, because currently this needs a vim.has() and some condition for backwards compatibility 🤔

[gegoune]

vim.version().minor >= 7 should work as well.

And I am not sure if using nordic namespace would apply global highlight, would it?

[andersevenrud]

I believe it will work exactly the same. The 0 argument is just a way to eliminate the need to create your own ns.

[gegoune]

Unless I am doing something wrong I am getting different results depending on whether 0 or custom namespace is used. It works with 0 but not with ns.

[andersevenrud]

I see. I'm not familiar with this specific highlighting function, but in hindsight that makes sense... it's the same for virtual text which I've used extensively in another plugin I work on. Derp!

[gegoune]

I am working on some refactor to nordic to work with that new interface. Will get it to working state and open PR.

[andersevenrud]

Since vim.highlight.create now emits a deprecation warning on main neovim branch, I added basic support for the new API.

eb096c0