search

nwolverson / purescript-language-server

MIT License
184 stars 42 forks source link

Add documentation to hover #25

Closed Krzysztof-Cieslak closed 6 years ago

Krzysztof-Cieslak commented 6 years ago

Old version: screenshot from 2018-02-26 01-05-11 screenshot from 2018-02-26 01-05-13 screenshot from 2018-02-26 01-05-16

New version (including clickable link if docs contains URL): screenshot from 2018-02-26 01-04-35 screenshot from 2018-02-26 01-04-30

screenshot from 2018-02-26 01-04-52

nwolverson commented 6 years ago

Nice! I guess the experience from Ionide is that the larger tooltips aren't obtrusive? I've seen that mentioned as an issue in the past but I think at least vscode's hover UI is smooth enough for it to be a non-issue

Krzysztof-Cieslak commented 6 years ago

I’ve never had any negative comments about too much information in tooltip. We actually put even more info in there in Ionide - fully qualified name, assembly from each symbol is, resolved types of generic parameters - and people seems to like it.