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

Commas, angle brackets dismiss signature help in strings #29669

Open
DanielRosenwasser opened this issue Sep 4, 2018 · 3 comments
Open

Commas, angle brackets dismiss signature help in strings #29669

DanielRosenwasser opened this issue Sep 4, 2018 · 3 comments
Labels
Area-IDE Bug help wanted The issue is "up for grabs" - add a comment if you are interested in working on it
Projects
Milestone

Comments

@DanielRosenwasser
Copy link
Contributor

DanielRosenwasser commented Sep 4, 2018

signaturehelpcommasasretriggers

TL;DR

We believe that all of C#'s trigger characters should probably be considered retrigger characters.

Context

We (@jessetrinity, @sana-ajani, @amcasey and I on the TypeScript team) are trying to tackle the Visual Studio side of microsoft/TypeScript#1150. We noticed that the most "intuitive" implementation actually results in the undesirable behavior highlighted above. When checking to see what C# does, we saw the same behavior. Rather than diverging, we wanted to consult with Roslyn first to verify whether this is a bug.

Details

Version Used: Latest

Steps to Reproduce:

  1. Write a method Foo accepting a string
  2. Trigger signature help session within a string literal being passed to Foo
  3. Enter a comma character within the string (,)

Expected Behavior:

Signature help remains

Actual Behavior:

Signature help is dismissed

@jinujoseph jinujoseph added Bug help wanted The issue is "up for grabs" - add a comment if you are interested in working on it labels Sep 6, 2018
@jinujoseph jinujoseph added this to the Unknown milestone Sep 6, 2018
@sharwell sharwell added this to InQueue in Small Fixes Sep 12, 2018
@CyrusNajmabadi
Copy link
Member

is this still an issue for you @DanielRosenwasser ?

@DanielRosenwasser
Copy link
Contributor Author

It was more of a quirk that I thought Roslyn would be interested in fixing, but I'd understand if it's not worth the time.

@jasonmalinowski jasonmalinowski changed the title Commas dismiss signature help in strings Commas, angle brackets dismiss signature help in strings Mar 11, 2021
@jasonmalinowski
Copy link
Member

This impacts typing a < too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE Bug help wanted The issue is "up for grabs" - add a comment if you are interested in working on it
Projects
Small Fixes
  
InQueue
Development

No branches or pull requests

5 participants