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

Jump to bottom

x/tools/gopls: move signature on top of quick documention on hover #33550

Closed
inliquid opened this issue Aug 8, 2019 · 3 comments
Closed

x/tools/gopls: move signature on top of quick documention on hover #33550

inliquid opened this issue Aug 8, 2019 · 3 comments
Labels
Documentation FrozenDueToAge gopls Issues related to the Go language server, gopls.
Milestone
Unreleased

Comments

@inliquid
Copy link

inliquid commented Aug 8, 2019

As a result of #32561 gopls now has additional config parameter, which allows to return full docs on hover. However atm this makes it less useful as every time documentation provided has some decent amount of text, it's only possible to see the signature after scrolling to the very bottom.
изображение

Please move signature above the documentation, as it was done previously in sourcegraph's language server, bingo, and as it's in the default vscode-go config with godoc or gogetdoc.
@gopherbot gopherbot added this to the Unreleased milestone Aug 8, 2019
@gopherbot gopherbot added Documentation gopls Issues related to the Go language server, gopls. labels Aug 8, 2019
@gopherbot
Copy link

Change https://golang.org/cl/189459 mentions this issue: internal/lsp: move documentation below signature for hover

@stamblerre
Copy link
Contributor

I've just come to the realization that this is effectively a duplicate of #33352. I think this needs a little more thought than my quick CL indicates. Both orderings are potentially valuable, and this issue has led me think that offering configurations might be a viable approach here. I will close this issue in favor of #33352.

@stamblerre stamblerre mentioned this issue Aug 8, 2019
Closed
@inliquid
Copy link
Author

inliquid commented Aug 9, 2019

Sorry I don't understand, why don't just make it in same way, as it was done in all other versions listed above, (btw, including Goland quick documentation feature), signature at the top and the rest is scrollable documentation text?
For example here is how it looks like in VS Code when turning off gopls and with simple godoc:
изображение

@golang golang locked and limited conversation to collaborators Aug 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Documentation FrozenDueToAge gopls Issues related to the Go language server, gopls.
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
3 participants
@stamblerre @inliquid @gopherbot