-
Notifications
You must be signed in to change notification settings - Fork 46
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 inlay hints #48
Add inlay hints #48
Conversation
e247fbe
to
81032a7
Compare
81032a7
to
5792a7b
Compare
5792a7b
to
0653db4
Compare
So, I went down a rabbit hole and it turns out that the However VSCode uses a semi-transparent color for the background. Here is an alternative render using |
Thank you a lot for looking into it! Also thanks for providing screenshots for all options!
This looks nice, but in my opinion without the background it's a bit better. We probably better to leave the comment for this new color (that it's taken from VSCode not part actually a part of |
0653db4
to
347df53
Compare
I agree that without the background it looks less distracting and closer to VSCode, even if I quite like the Helix style. I've added a comment as you suggested! |
Add support for the highlight group
LspInlayHint
. This should fix #44.I chose arbitrary colors but I think they're quite close to those in the Helix theme.
Before:
After:
After, Mirage :