Skip to content
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

Hide completions when signature help is enabled #1891

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

fvclaus
Copy link
Contributor

@fvclaus fvclaus commented Sep 24, 2021

Signed-off-by: Frederik Claus f.v.claus@googlemail.com

Implementation for redhat-developer/vscode-java#2063

@eclipse-ls-bot
Copy link

Can one of the admins verify this patch?

Signed-off-by: Frederik Claus <f.v.claus@googlemail.com>
Signed-off-by: Frederik Claus <f.v.claus@googlemail.com>
@fvclaus fvclaus force-pushed the hide-completion-on-signature-help branch from 9209c3f to 4c20da2 Compare September 27, 2021 13:42
@fvclaus
Copy link
Contributor Author

fvclaus commented Oct 11, 2021

@Eskibear can you please look at the code and decide if we can move forward with it or not?

@fbricon
Copy link
Contributor

fbricon commented Oct 12, 2021

This doesn't seem to have an impact in vscode. I can see the filter text is simplified, but it still shows all completions, shows and inserts all parameters for an overloaded method.

@fbricon
Copy link
Contributor

fbricon commented Oct 12, 2021

add to whitelist

@fvclaus
Copy link
Contributor Author

fvclaus commented Oct 14, 2021

@fbricon When the filter text is used, vscode will hide completions once the ( is entered:
jdt_ls_hide_completions
Without the filter text, vscode will show the completions that match the text being entered:
jdt_ls_show_completions

@fvclaus
Copy link
Contributor Author

fvclaus commented Oct 14, 2021

For this to work, the signatureHelp needs to be active:

"java.signatureHelp.enabled": true,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants