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

Update manual for inlay hint customization #11708

Merged
merged 1 commit into from
Mar 15, 2022

Conversation

ian-h-chamberlain
Copy link
Contributor

@ian-h-chamberlain ian-h-chamberlain commented Mar 14, 2022

Related to #6405, #6883 but not sure if they should be closed or not as this is just a documentation update.

This functionality was changed as of #11445 and now can be customized using native VSCode settings instead of rust-analyzer-specific ones.

Closes rust-lang#6883 

This functionality was changed as of rust-lang#11445 and now can be customized using native VSCode settings instead of `rust-analyzer`-specific ones.

// Overrides for specific kinds of inlay hints
"rust_analyzer.inlayHints.foreground.typeHints": "#fdb6fdf0",
"rust_analyzer.inlayHints.foreground.paramHints": "#fdb6fdf0",
"rust_analyzer.inlayHints.background.chainingHints": "#6b0c0c81"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far as I can tell, there is no equivalent for the chainingHints color available in VScode settings yet – they just use the base editorInlayHint.* colors.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ye the proposal only defines type and parameter hints, everything else will fall back to the general

@Veykril
Copy link
Member

Veykril commented Mar 15, 2022

Thanks!
bors r+

@bors
Copy link
Contributor

bors bot commented Mar 15, 2022

@bors bors bot merged commit 7308b3e into rust-lang:master Mar 15, 2022
@ian-h-chamberlain ian-h-chamberlain deleted the patch-1 branch March 15, 2022 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants