Inconsistency in Constructor tooltips

[tldrlol]

Windows 10, vanilla VS 2017 install.

Given this (non-compiling code):

type Foo() =
  new(_: int) = Foo ()
  new(_: string) = Foo ()

System.String
Foo

There is an inconsistency between the tooltips generated over the System.String line, and the one for Foo.

To illustrate:

[ReedCopsey]

Given that we can now treat ctor as a function (ie: you can pipe into a ctor, etc), it seems like String should definitely be new : char [] -> String and similar in order to make sense from a discoverability POV.

[matthid]

Just to add, this would probably fix some open bugs in FSharp.Formatting as well

[dsyme]

Yes, I agree

[dsyme]

I presume this issue is around here https://github.com/Microsoft/visualfsharp/blob/44fa027b308681a1b78a089e44fa1ab35ff77b41/src/fsharp/NicePrint.fs#L1302

I think we should use this instead https://github.com/Microsoft/visualfsharp/blob/44fa027b308681a1b78a089e44fa1ab35ff77b41/src/fsharp/MethodCalls.fs#L589

[vasily-kirichenko]

Fixed in #3754

[dsyme]

@vasily-kirichenko Thanks for fixing this!