-
Notifications
You must be signed in to change notification settings - Fork 182
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
Rework textDocument/signatureHelp #1513
Conversation
- Proper handling of markdown parameter docs. - Don't assume that function signatures have an `(` and `)`. - Put the *active parameter docs* at the top, font-size 1rem. - Put the *signature docs* under that, font-size 0.9rem. - Use 0.9rem font-size for the intro message. - KISS.
9c9e5e5
to
c7d10e1
Compare
When I've triggered signature help manually and gotten this response from the server: {
"activeParameter": null,
"activeSignature": null,
"signatures": []
} it opened a very tiny, empty popup. |
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.
It seems to work as well as before on few basic tests so I'll not dig too much into the code :)
Are you planning some further improvements to signature help in the near future? I think it works poorly in the sense that it disappears way too easily on writing the code or moving the cursor around. So if you are not planning work then I might look into it eventually.
this is actually invalid. "activeParameter" and "activeSignature" are supposed to be an integer. The spec also states that if there are no signatures the server should return |
No. Feel free to improve it. Though I feel that you'd be swimming against the ST API stream. There is no natural way to keep a popup open when typing. |
Could |
(
and)
.close #1001