mfussenegger / nvim-jdtls

Extensions for the built-in LSP support in Neovim for eclipse.jdt.ls
GNU General Public License v3.0
1k stars 62 forks source link

java_type_hierarchy: class, method type hierarchy on the quickfix list #512

Closed lbrayner closed 11 months ago

lbrayner commented 1 year ago

java_type_hierarchy uses java.navigate.openTypeHierarchy and java.navigate.resolveTypeHierarchy to show type hierarchy information of a class or method on the quickfix list.

lbrayner commented 1 year ago

Please consider adding something like this to nvim-jdtls.

lbrayner commented 1 year ago

The motivation for this was https://github.com/eclipse/eclipse.jdt.ls/issues/2684.

I was looking for a way to jump from an interface method implementation on a class to its declaration on the interface file. java_type_hierarchy also delivers that..

mfussenegger commented 1 year ago

Type hierarchy is also part of the spec: https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#textDocument_prepareTypeHierarchy

I think eclipse.jdt.ls and neovim core should implement that instead of going with the custom off-spec variant

mfussenegger commented 11 months ago

See also https://github.com/eclipse-jdtls/eclipse.jdt.ls/pull/2769

I'm not going to merge the off-spec variant. type hierarchy should be implemented in neovim core instead