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

x/tools/gopls: decide if import statements should have definitions or links #39065

Closed
stamblerre opened this issue May 14, 2020 · 1 comment
Closed
Labels
FrozenDueToAge gopls Issues related to the Go language server, gopls. Tools This label describes issues relating to any tools in the x/tools repository.
Milestone

Comments

@stamblerre
Copy link
Contributor

See microsoft/vscode-go#2614 for more context.

Ctrl + Click in VS Code is a shortcut for both jump to definition and opening a link. We provide links to https://pkg.go.dev for import statements, as well as definitions in the form of opening a window that shows the files in the package. The behavior is unintuitive because 2 things happen at once.

Currently, users can disable document links through the go.experimentalLanguageServerFeatures setting. Based on the response to microsoft/vscode#81520, my guess is that VS Code will not be fixing this any time soon, and even if they did, the only correct fix would be to change one of their shortcuts. Users are probably used to things working a specific way, so I doubt they would change a default shortcut, particularly since, in the majority of cases, definitions and document links don't collide.

I propose that we create a gopls setting to choose between links and definitions for import statements. The default could be both. This way, users will not lose document links for other cases in which there are no collisions.

@stamblerre stamblerre added this to the gopls/v0.5.0 milestone May 14, 2020
@gopherbot gopherbot added Tools This label describes issues relating to any tools in the x/tools repository. gopls Issues related to the Go language server, gopls. labels May 14, 2020
@gopherbot
Copy link

Change https://golang.org/cl/242457 mentions this issue: internal/lsp: add an importShortcut configuration

@stamblerre stamblerre modified the milestones: gopls/v0.5.0, gopls/v0.4.4 Jul 16, 2020
@golang golang locked and limited conversation to collaborators Jul 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge gopls Issues related to the Go language server, gopls. Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Development

No branches or pull requests

2 participants