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

Semantic highlighting for function exit points #4691

Closed
lnicola opened this issue Jun 1, 2020 · 9 comments · Fixed by #9375
Closed

Semantic highlighting for function exit points #4691

lnicola opened this issue Jun 1, 2020 · 9 comments · Fixed by #9375
Labels
E-hard S-actionable Someone could pick this issue up and work on it right now

Comments

@lnicola
Copy link
Member

lnicola commented Jun 1, 2020

As seen in IntelliJ

@matklad
Copy link
Member

matklad commented Jul 11, 2020

Yessss I want this a lot. The problem is, I don't see an LSP request we can use for it :0(

Maybe abuse document highlight?

@matklad matklad added the E-hard label Jul 11, 2020
@matklad
Copy link
Member

matklad commented Jul 21, 2020

image

@kjeremy
Copy link
Contributor

kjeremy commented Jul 21, 2020

That takes in a position though. You couldn't do this with semantic highlighting and token modifiers?

@matklad
Copy link
Member

matklad commented Jul 21, 2020

@kjeremy that's exatcly the UI -- higlights of all exit points activates when you are on a single exit point (and probably it makes sense to trigger it on return type as well).

@kjeremy
Copy link
Contributor

kjeremy commented Jul 21, 2020

Ah! I thought it was always active. This would be really nice to have.

@woody77
Copy link
Contributor

woody77 commented Jul 21, 2020

always active is definitely doable, but it can't be themed as a background change (but could be given bold or other font styling).

In the theme that I'm working on, making the ? operator extra clear is one of my goals (as well as other subtle, easy-to-miss things).

@kjeremy
Copy link
Contributor

kjeremy commented Aug 16, 2020

I think we could just use DocumentHIghlightKind::Text for this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-hard 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.

4 participants