add minimal data model to config so that palette param gets completions

[jameswalls]

Simplify the config experience of the on_highlight, on_palette, and after_palette config functions by adding code completion to the palette parameter.

[AlexvZyl]

Thanks! That is nice. Are those generated or did you do them manually?

[jameswalls]

Thanks! That is nice. Are those generated or did you do them manually?

The screenshot is real :) It’s the actual output of cmp through the lsp.

[5-pebbles]

Thanks! That is nice. Are those generated or did you do them manually?

The screenshot is real :) It’s the actual output of cmp through the lsp.

I think he meant is the types.lua file automatically generated in some way.

[jameswalls]

I think he meant is the types.lua file automatically generated in some way.

Oh got it, it’s manual.

[5-pebbles]

Alright so there seems to be a better way of doing this... I am still figuring how it works but would it be alright if I push the change to this PR? You define the types above the values and the lsp will fill in all the fields it's self. I would just make a review suggestion but it takes a lot of small changes in different files.

Also thanks for pointing this out it will make configuring the extended palette and highlights much easier.

[5-pebbles]

And one other thing I can get it working when editing the nordic source code but I can't get anything when working in my config... It won't work for me when using your system or my new one, does it still work for you when you run it remotely:

    "jameswalls/nordic.nvim",
    branch = "palette_completions",

[jameswalls]

... would it be alright if I push the change to this PR?

sure! go for it.

does it still work for you when you run it remotely:

Yes it works for me:

Try commenting out all the config, then manually clean the plugin, and then install it again.

[5-pebbles]

I still can't get it working with my config, but I tried using yours, and it works like a charm!

I inlined the classes, and lua-ls should now automatically generate all the fields. When you have the time can you give it a test and if it's all working as expected I will get this merged.

[jameswalls]

Very nice! All three work for me, including the highlights!