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
CHE-5334. Fix hover state #5899
Conversation
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/3275/ |
@@ -363,7 +365,18 @@ public boolean canDo(InitializedLanguageServer element) { | |||
@Override | |||
public boolean handleResult(InitializedLanguageServer element, Hover hover) { | |||
if (hover != null) { | |||
result.getContents().addAll(hover.getContents()); | |||
List<Either<String, MarkedString>> contents = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a constructor that makes a HoverDto out of a Hover in the generated HoverDto class. It may be simpler and safer to use that one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tsmaeder fixed, thank you!
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/3295/ |
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
8062061
to
8cd2ea4
Compare
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/3311/ |
What does this PR do?
Fix hover state
What issues does this PR fix or reference?
#5334
Changelog
Fix hover state
Signed-off-by: Roman Nikitenko rnikiten@redhat.com