shaunsingh / nord.nvim

Neovim theme based off of the Nord Color Palette, written in lua with tree sitter support
GNU General Public License v2.0
796 stars 107 forks source link

Recent updates to treesitter cause everything to be 'glacier' #143

Open tomit4 opened 1 year ago

tomit4 commented 1 year ago

nord_issue nord_issue

I don't know if this is something you can fix, but as soon as my linter turns on, all that was previously white is now converted to the glacier blue. The contrast makes it hard to read my code. I love your theme, could you please help?

Here's my neovim config if that helps:

https://github.com/tomit4/notes/tree/main/.config/nvim

Thanks for any help you can possibly give in advance.

shaunsingh commented 1 year ago

Thanks, I'll fix it

folliehiyuki commented 1 year ago

This is caused by LSP semantic highlight groups (not set by nord.nvim) override the defined treesitter groups (as they have higher priority).

Related to https://github.com/shaunsingh/nord.nvim/issues/134.

ZakariaBouzouf commented 1 year ago

The same here !

mvsrgc commented 1 year ago

Same image

tomit4 commented 1 year ago

Hi all. Just fyi, I was able to find a temporary solution to this problem. Apparently Neovim's latest update to 0.9.0 includes using LSP semantic tokens (I only know about them because I posted a quickly closed issue on the official Neovim official github issues board). They didn't solve the problem for me but simply told me to type :h lsp-semantic_tokens.

I also browsed around on reddit and found this lua snippet that turns LSP semantic tokens off and returned my colors to their original state. Hopefully this helps until this theme includes them? It is also possible this is an issue with treesitter, but I uninstalled treesitter while troubleshooting and the problem persisted.

` -- turns off LSP semantic tokens by default vim.api.nvim_create_autocmd("LspAttach", { callback = function(args) local client = vim.lsp.get_client_by_id(args.data.client_id) client.server_capabilities.semanticTokensProvider = nil end, }) ``

KubaBaniak commented 11 months ago

Same problem here

Code snippet from @tomit4 fixed this.

azthec commented 10 months ago

I have a similar issue and I have not been able to fix this with the snippet above, it is specific to this theme as I checked.

image image
shaunsingh commented 10 months ago

I'll try to add semantic token support to fix the issue, thanks for letting me know and sorry for the delay