-
Notifications
You must be signed in to change notification settings - Fork 61
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
LspShowImpl alternative to LspPeekImpl #574
Comments
After posting the request I decided to have a closer look. It seems to be as simple as this:
Is that right? Or does it need anything else? |
When you can have multiple locations where a symbol is referenced or defined, then using a quickfix or location list makes sense. The LspShowImpl command will show only one location where a symbol is implemented. So the quickfix list will have only one entry? |
Hmm. Maybe it depends on the language the LSP is for. I'm primarily interested in Go and so I'm using gopls. In my case, the proposed LspShowImpl command shows many locations in the quickfix. |
I see that we already have the "LspGotoImpl" command which provides the exact same functionality. Can you use that command? |
Ah. I didn't spot that. I was too busy looking at the Show group of commands. LspGotoImpl is exactly what I need. Thanks. |
Would it be possible for a
LspShowImpl
command be added?There is a
LspPeekImpl
command but I prefer how theLspShowReferences
command works as compared to the peek commands. Peek is good but showing the list in the quickfix suits my workflow better.The text was updated successfully, but these errors were encountered: