Skip to content
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

Show potential fixes link does not work #27168

Closed
jinujoseph opened this issue May 25, 2018 · 3 comments
Closed

Show potential fixes link does not work #27168

jinujoseph opened this issue May 25, 2018 · 3 comments

Comments

@jinujoseph
Copy link
Contributor

jinujoseph commented May 25, 2018

Ported from link
Affected Branch: [27710.0.d15.8]
Affected Build: 15.8.0 Preview 2.0
Affected Product Language:

Steps to reproduce:

  1. Create new Class Library (.NET Framework) project
  2. Add following code to Class:
public Int32[,] Properties { get; set; } 

public void Foo() 

{ 

    var list = new List<string>(); 

    list.Add("dsfds"); 

  

    var a = DateTime.Now.ToString("MM"); 

} 

  

public string ToString() 

{ 

    return "test"; 

} 
  1. Hover mouse over ToString()

  2. Click on "Show potential fixes" link or click on light bulb near ToString or press Alt+Enter or Ctrl+.

Expected: context menu with fix variants appeared

Actual: context menu with fix variants is not appeared

Note: click on light bulb on the left side of editor works

You can also install Microsoft.CodeAnalysis.FxCopAnalyzers NuGet package to get more warnings (which also does not work)

Repro Gif

@jinujoseph jinujoseph added this to the Unknown milestone May 25, 2018
@jinujoseph
Copy link
Contributor Author

Affected Branch: [27911.0.d15.8]
Affected Build: 15.8.0 Preview 5.0
Affected Product Language:

Steps to reproduce:

  1. Create Class Library C# project
  2. Add Class1:
    public abstract class Class1
    {
    [Obsolete]
    public abstract void Foo();
    }
  3. Add Class2:
    class Class2 : Class1
    {
    public override void Foo()
    {
    throw new NotImplementedException();
    }
    }
  4. Hover mouse over underlined Foo
  5. Click on Show potential fixes

Expected: menu with option "add the Obsolete attribute to method" appears
Actual: nothing happens

  1. Place cursor to underlined Foo
  2. Press Alt+Enter or Ctrl+.

Expected: option "add the Obsolete attribute to method" is available
Actual: option "add the Obsolete attribute to method" is not available

Repro Gif

@v-rodudi
Copy link

JFYI: ticket author @v-rodudi won't be available after 9/4 so please ask repro-related questions before this date if they are.

@CyrusNajmabadi
Copy link
Member

Closing out as this works fine for me locally. Likely an old issue long since fixed.

@CyrusNajmabadi CyrusNajmabadi closed this as not planned Won't fix, can't repro, duplicate, stale Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants