Combine error tooltip with quick info

microsoft / TypeScript-Sublime-Plugin

IO wrapper around TypeScript language services, allowing for easy consumption by editor plugins

Apache License 2.0

1.72k stars 235 forks source link

Combine error tooltip with quick info #562

Closed zhengbli closed 7 years ago

zhengbli commented 7 years ago

We learned from user feedback that the initial implementation of the inline error tooltip is annoying and gets in the way of typing. This PR changes the implementation to show the error tip during mouse hovering, which is the same behavior with VSCode and VS.

Now the quick info hover tooltip is combined with the error tooltip, so if there is no error in the position, only quick info should be shown: otherwise both will be shown:

roblav96 commented 7 years ago

@zhengbli I like this one :D

lindelleric commented 7 years ago

+1 :D

DanielRosenwasser commented 7 years ago

Looks good apart from what I can see (apart from nits).

serhiipalash commented 6 years ago

@zhengbli is it possible to combine TypeScript tooltip with SublimeLinter TSlint tooltip?

In VS Code they are combined in one tooltip.