Skip to content

Update string text for keybindings reset.#24313

Merged
333fred merged 3 commits intodotnet:dev15.6.xfrom
333fred:update-keybinding-text
Jan 20, 2018
Merged

Update string text for keybindings reset.#24313
333fred merged 3 commits intodotnet:dev15.6.xfrom
333fred:update-keybinding-text

Conversation

@333fred
Copy link
Copy Markdown
Member

@333fred 333fred commented Jan 18, 2018

Ask Mode template completed

Customer scenario

When a customer disables an extension that messes up keybindings, the displayed notification can cause confusion as to what actually broke the bindings.

Bugs this fixes

N/A

Workarounds, if any

None

Risk

Little. This just updates the text of a gold bar and adds a call to string.Format.

Performance impact

None. Only new code is a call to string.Format.

Is this a regression from a previous update?

N/A

Root cause analysis

N/A

How was the bug found?

N/A

Test documentation updated?

N/A

@333fred 333fred requested a review from a team as a code owner January 18, 2018 18:47
@333fred 333fred requested review from jinujoseph and kuhlenh January 18, 2018 18:47
…g-text

* dotnet/dev15.6.x:
  Fix symbol completion after 'in' (dotnet#24335)
  use PascalCase for const name
  Limit compiler server pipe name length (dotnet#24265)
  Test ConvertedType on LHS of deconstruction-assignment (dotnet#24158)
  remove unused usings
  use .editorconfig files
  address more comments
  address code review comments
  move newly added text into resource file
  add text and hyperlink to C# code style page
@333fred
Copy link
Copy Markdown
Member Author

333fred commented Jan 19, 2018

Ping @Pilchie for merge approval.

@Pilchie
Copy link
Copy Markdown
Member

Pilchie commented Jan 19, 2018

Approved.

@333fred 333fred merged commit ea75a53 into dotnet:dev15.6.x Jan 20, 2018
@333fred 333fred deleted the update-keybinding-text branch January 20, 2018 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants