[Roslyn] Add support for constructor/method signature intellisense

[jtbm37]

Roslyn has support for it here and has tests.

We just need to show the signature when typing (.

Optionally user can call the function when point is inside parentheses.

[jtbm37]

We should also probably show it when typing a , inside parentheses but also when there is only one opening bracket.

[mikavilpas]

👍
What do you think, would eldoc suffice? We display current type information with eldoc.

On the other hand, for example lispy goes the extra step and displays stuff like this in an overlay:

[jtbm37]

I am thinking we should use company so we can expand the selection like a snippet and then filling in each parameters by tabbing through.
We could even use company quickhelp to display extra information.

I am not sure how eldoc would display 5 constructor/method overloads.

[mikavilpas]

Ah I see what you mean. Eldoc is clearly out of the question.
I haven't really looked at the server api yet.

2016-05-11 1:12 GMT+03:00 jtbm37 notifications@github.com:

I am thinking we should use company so we can expand the selection like a
snippet and then filling in each parameters by tabbing through.
We could even use company quickhelp
https://github.com/expez/company-quickhelp to display extra information.

I am not sure how eldoc would display 5 constructor/method overloads.

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#224 (comment)

[jtbm37]

Roslyn has now a new completion API available. This is very exciting.

It needs to be implemented in omnisharp-roslyn. When it is, we could have new completion capabilities like in object initializer which we previously did not.

[mikavilpas]

This is exciting!
Let's keep an eye on that. I wonder if the completion API will change, or if it's just going to be an internal change with no visible changes to the outside.

[jtbm37]

How is that looking ?

[mikavilpas]

[willl]

@jtbm37 Completion API has now been merged into omnisharp-roslyn: OmniSharp/omnisharp-roslyn#583 :)

[jtbm37]

@willl Thanks. I will surely be testing that fairly soon. 👍