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

Add option to disable hover documentation #9232

Closed
bitboxx opened this issue Jun 12, 2021 · 1 comment · Fixed by #9264
Closed

Add option to disable hover documentation #9232

bitboxx opened this issue Jun 12, 2021 · 1 comment · Fixed by #9264
Labels
S-actionable Someone could pick this issue up and work on it right now

Comments

@bitboxx
Copy link

bitboxx commented Jun 12, 2021

The hover api documentation are often useless as they can be long (while shown on a small hover pop-up window) and in the way of error / warning messages that are more relevant.

This pop-up window is meant for a short contextual information, rather than a full API documentation.

No doubt some people like this feature, but please add an option to disable the documentation hover. I've tried toggling on/off the Rust Analyzer hover settings, nothing worked.

image

@Veykril Veykril added the S-actionable Someone could pick this issue up and work on it right now label Jun 12, 2021
@lf-
Copy link
Contributor

lf- commented Jun 16, 2021

The diagnostics buried under stuff is a long-standing bug in VSCode, unfortunately :/
microsoft/vscode#73120

bors bot added a commit that referenced this issue Jun 21, 2021
9264: feat: Make documentation on hover configurable r=Veykril a=Veykril

This also implements deprecation support for config options as this renames `hoverActions_linksInHover` to `hover_linksInHover`.

Fixes #9232

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
bors bot added a commit that referenced this issue Jun 21, 2021
9264: feat: Make documentation on hover configurable r=Veykril a=Veykril

This also implements deprecation support for config options as this renames `hoverActions_linksInHover` to `hover_linksInHover`.

Fixes #9232

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
@bors bors bot closed this as completed in 25bf451 Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-actionable Someone could pick this issue up and work on it right now
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants