-
-
Notifications
You must be signed in to change notification settings - Fork 378
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
Code lens to display type signature for local bindings in where clauses #2943
Comments
Configuration is bad! I say have it be enabled whenever the main type signature lens is. |
Could you please explain why configuration is bad? Ideally I want to control everything displayed on my screen by category... |
Because it means more things for users to think about, a bigger settings page, and more things for us to support. I think if we can, it's best only to introduce configuration in cases where there's a real split in what people want and people are annoyed if they don't get the one they want. |
So, we can implement this feature into ghcide? |
I think that's reasonable. Especially since I'd hope the code is mostly there already! |
@July541 Now I got it, I misunderstood that "code lens" is, I thought lens is a bubble. That is cool. But why this issue is concerned only on where clauses? Should be cool to see this in let clauses too. Now I do not get them. |
I think doing a code lens is good: code lenses fit better for places where you can put signatures on their own lines, which is probably better in general. |
Reminded by #2019 (comment), I think it would be great to have a configurable plugin to display a signature code lens for local bindings in where clauses.
Please let me know if there has more places need code lenses.
Edited: I'll try to pick bindings fromShould be TypeCheckedSource.RenamedSource
, please let me know if there is a more efficient way.The text was updated successfully, but these errors were encountered: