search

doom-neovim / doom-nvim

A Neovim configuration for the advanced martian hacker
GNU General Public License v2.0
1.01k stars 107 forks source link

[BUG] Jump to definition bind from `<leader> c d` is overridden by diagnostics #427

Open robclancy opened 1 year ago

robclancy commented 1 year ago

I just spent a fair while thinking the language server didn't work or I'm not understanding something because I don't see a "Jump to Definition" option. Eventually using the gd I found it was fine but I was still confused why <leader> c wouldn't have it as an option.

Well you do, but it is overridden by diagnostics.

https://github.com/doom-neovim/doom-nvim/blob/main/lua/doom/modules/features/lsp/init.lua#L355

https://github.com/doom-neovim/doom-nvim/blob/main/lua/doom/modules/features/lsp/init.lua#L368

Diagnostics should probably be set to something else.

Also pressing g doesn't give any of the LSP stuff in the help there either, but I assume that is by design.