-
Notifications
You must be signed in to change notification settings - Fork 138
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
Sideline displays full info, including docs #292
Comments
Generally, onHover request is not designed for what we are trying to use it for(sideline) and it does not offer good experience for some of the servers. There are 3 possibilities:
|
I see, I think 2 is the most promising. By first rendering the full contents (with |
We used to return the Where MarkedString is defined as follows:
/**
* MarkedString can be used to render human readable text. It is either a markdown string
* or a code-block that provides a language and a code snippet. The language identifier
* is semantically equal to the optional language identifier in fenced code blocks in GitHub
* issues. See https://help.github.com/articles/creating-and-highlighting-code-blocks/#syntax-highlighting
*
* The pair of a language and a value is an equivalent to markdown:
* ```${language}
* ${value}
* ```
*
* Note that markdown strings will be sanitized - that means html will be escaped.
* @deprecated use MarkupContent instead.
*/
type MarkedString = string | { language: string; value: string }; So this will have to be fixed properly some time. |
lsp-ui-sideline
displays way too long lines, apparently trying to include the full text, the same as it shows inlsp-ui-doc
:I'm using
Here are the full logs of the server:
Here's the request/response pair for the
return
:#264 seems to be relevant.
Ping @yyoncho @alanz
The text was updated successfully, but these errors were encountered: