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

#5750 fix cursor position after completion proposal inserted #6020

Merged
merged 3 commits into from
Aug 19, 2017

Conversation

evidolob
Copy link
Contributor

What does this PR do?

fix cursor position after completion proposal inserted

What issues does this PR fix or reference?

#5750

Signed-off-by: Yevhen Vydolob <evidolob@codenvy.com>
@benoitf
Copy link
Contributor

benoitf commented Aug 17, 2017

IMHO test would be great as I saw many times bugs around cursor position after code completion.

@codenvy-ci
Copy link

Yevhen Vydolob added 2 commits August 19, 2017 11:11
…/che into fixCursorPosition

Signed-off-by: Yevhen Vydolob <evidolob@codenvy.com>

# Conflicts:
#	plugins/plugin-languageserver/che-plugin-languageserver-ide/src/main/java/org/eclipse/che/plugin/languageserver/ide/editor/codeassist/CompletionItemBasedCompletionProposal.java
Signed-off-by: Yevhen Vydolob <evidolob@codenvy.com>
@codenvy-ci
Copy link

@evidolob evidolob merged commit 93eaf18 into master Aug 19, 2017
@evidolob evidolob deleted the fixCursorPosition branch August 19, 2017 13:03
@slemeur slemeur added the kind/bug Outline of a bug - must adhere to the bug report template. label Sep 28, 2017
@slemeur slemeur added this to the 5.17.0 milestone Sep 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants