-
-
Notifications
You must be signed in to change notification settings - Fork 812
LS: hover info for custom types and their constructors #4451
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
Comments
Ah, I think that may be a bug. I'm sure I've used that feature before. I'll investigate |
I've tested every tagged release dating back to v1.0.0, and none of the versions provided hover info on my machine. Feel free to investigate further, but I'd like to work on this in the meantime! |
Go ahead! I haven't had a chance to look at it yet |
Ran into a small issue with implementing the proposed behavior because //// wibble.gleam
type Wibble {
// ^ Wibble
Wibble(Int)
// ^ Wibble(Int)
Wobble(String)
// ^ Wobble(String)
} I think this is also more readable and useful. |
While working on this, I've also realized that type aliases do not have any hover info. Is this something that we want to support? If so, should I open a separate issue or merge that into this one? |
The language server currently doesn't return hover info for custom types and their constructors. It would be useful for the language server to support this.
The proposed behavior above would be consistent with the hover info of type annotations and constructor expressions.
The text was updated successfully, but these errors were encountered: