-
Notifications
You must be signed in to change notification settings - Fork 677
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
Completion list incorrectly includes full signatures for members #1491
Comments
Unfortunately, I'm unable to reproduce this. I tried the following:
Could you clarify your repro steps? Thanks! |
@DustinCampbell very strange behaviour, btw. Looks like it depends on base class methods. I recorded video with issue (it raised only on first attempt). |
@DustinCampbell in my case happens it: I write |
@Leopotam: Thanks for the video. That is very strange behavior. 😄 I notice that you're using VS Code Insiders. I wonder if the issues is there. I'll install and take a look. @bernardbr: The behavior you described is different than what @Leopotam is seeing. Note that in your example there is code after the "if". |
Yeah, I realized that now ... I'm sorry. |
@DustinCampbell class was created in unity3d solution - its why i can used UnityEngine.MonoBehaviour class. All methods at popup list are valid methods of MonoBehaviour type. |
@Leopotam: I don't have Unity set up to precisely repro. Do you see the same problem on the RTM version of VS Code? |
@DustinCampbell Sorry about my first comment. This behavior wasn't of Omnisharp. I disabled all other extensions and the problems gone. Thanks for your attention! |
@bernardbr: Glad to hear it -- thanks @bernardbr! |
@DustinCampbell i uploaded minimal unity3d-solution, you can try to reproduce issue on it. |
Thanks! I can totally repro with this solution. |
OK. I can repro with a smaller code example: class Base
{
public void Invoke(float value) { }
}
class Derived : Base
{
public void M()
{
// if here
}
} |
OK. I see what's wrong. |
Environment data
dotnet --info
output:VS Code version:
1.13.0-insider (1.13.0-insider) 3bb9b5314ef50bd8225aba0565cadc6ca19bc821 2017-05-18T06:18:32.269Z
C# Extension version:
1.10.0-beta3 dc1908d
Steps to reproduce
Expected behavior
Autocomplete with closing pair of parenthesis ")", or nothing. Even I try to insert "IF"-snippet with tab-key - it will do it only on second attempt (first one is just closing autocomplete popup). Totally not usable when you write condition expression in fast manner.
Actual behavior
Automatic insertion of first item at autocomplete list.
The text was updated successfully, but these errors were encountered: