-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ItemDefaults client capability is not implemented #741
Comments
There is support for itemDefaults, see https://github.com/eclipse/lsp4e/blob/92abe97f21f72dca1f07bb6e16fa452b1763c272/org.eclipse.lsp4e/src/org/eclipse/lsp4e/operations/completion/LSContentAssistProcessor.java#L230 and #599 . Can you please elaborate what is missing? |
The language server needs to know if the LSP client support Item defaults to generate completion with default. In other words it misses the LSP4E completion client capability, |
ah ok, thanks. I'm patchingit. |
And LSP4E don't support item default with editRange which IMHO is the most used to avoid generating a big JSON with the same edit range. See LSP4IJ https://github.com/redhat-developer/intellij-quarkus/blob/a0e94debf4d88d0866ab081491f8fe81ae547304/src/main/java/com/redhat/devtools/intellij/lsp4ij/operations/completion/LSPCompletionContributor.java#L148 |
To check that it is working, I suggest that you open completion with XML, it should return completion list with default editRange and not an editRange per completion item. |
We have supported ItemDefaults with our XMLLanguage Server to speed the completion for lemmix maven which can return big list (artifact completion) but we cannot benefit with feature since LSP4E client doesntsend thepropre completion ItemDefaults capability.
In vscode it send:
The text was updated successfully, but these errors were encountered: