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

Inconsistent autocompletion behavior for constructors #143

Closed
fbricon opened this issue Jan 19, 2017 · 2 comments
Closed

Inconsistent autocompletion behavior for constructors #143

fbricon opened this issue Jan 19, 2017 · 2 comments

Comments

@fbricon
Copy link
Collaborator

fbricon commented Jan 19, 2017

When triggering autocompletion on a constructor, when one continues typing, the selected/relevant proposal is lost, as some leading characters seem to be missing from the selection.

Additionally, in some circumstances, eg. after triggering autocomplete then deleting the previous character, the injected text is missing the leading characters (one or many).

See https://www.screencast.com/t/17vjLA8vC0hq

@fbricon
Copy link
Collaborator Author

fbricon commented Jan 19, 2017

@othomann do you a similar behavior in Orion? I wonder if this is a server or client problem (my guts say the latter)

@othomann
Copy link

No, I don't think I ever seen anything like this in Orion.

@fbricon fbricon changed the title Inconsistent autocompletion behavior Inconsistent autocompletion behavior for constructors Feb 2, 2017
fbricon added a commit to fbricon/eclipse.jdt.ls that referenced this issue Feb 2, 2017
Merge proposal `()` and required proposal `constructor` in 1 text edit
Fixes redhat-developer/vscode-java#143

Requires eclipse-jdtls#129 to be applied.

Signed-off-by: Fred Bricon <fbricon@gmail.com>
@fbricon fbricon self-assigned this Feb 2, 2017
@fbricon fbricon added this to the Late January 2017 milestone Feb 2, 2017
fbricon added a commit to fbricon/eclipse.jdt.ls that referenced this issue Feb 3, 2017
Merge proposal `()` and required proposal `constructor` in 1 text edit
Fixes redhat-developer/vscode-java#143

Requires eclipse-jdtls#129 to be applied.

Signed-off-by: Fred Bricon <fbricon@gmail.com>
fbricon added a commit to fbricon/eclipse.jdt.ls that referenced this issue Feb 3, 2017
Merge proposal `()` and required proposal `constructor` in 1 text edit
Fixes redhat-developer/vscode-java#143

Requires eclipse-jdtls#129 to be applied.

Signed-off-by: Fred Bricon <fbricon@gmail.com>
fbricon added a commit to fbricon/eclipse.jdt.ls that referenced this issue Feb 3, 2017
Merge proposal `()` and required proposal `constructor` in 1 text edit
Fixes redhat-developer/vscode-java#143

Requires eclipse-jdtls#129 to be applied.

Signed-off-by: Fred Bricon <fbricon@gmail.com>
fbricon added a commit to fbricon/eclipse.jdt.ls that referenced this issue Feb 3, 2017
Merge proposal `()` and required proposal `constructor` in 1 text edit
Fixes redhat-developer/vscode-java#143

Requires eclipse-jdtls#129 to be applied.

Signed-off-by: Fred Bricon <fbricon@gmail.com>
fbricon added a commit to fbricon/eclipse.jdt.ls that referenced this issue Feb 6, 2017
Merge proposal `()` and required proposal `constructor` in 1 text edit
Fixes redhat-developer/vscode-java#143

Requires eclipse-jdtls#129 to be applied.

Signed-off-by: Fred Bricon <fbricon@gmail.com>
fbricon added a commit to eclipse-jdtls/eclipse.jdt.ls that referenced this issue Feb 6, 2017
Merge proposal `()` and required proposal `constructor` in 1 text edit
Fixes redhat-developer/vscode-java#143

Requires #129 to be applied.

Signed-off-by: Fred Bricon <fbricon@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants