Skip to content

Conversation

@PankajBhojwani
Copy link
Contributor

Summary of the Pull Request

Fixes a regression from the actions MVVM change in #14292 - attempting to overwrite a keybinding was displaying a warning but propagating the change before the user acknowledged it.

Validation Steps Performed

The overwrite key binding warning in the SUI works like before

PR Checklist

@microsoft-github-policy-service microsoft-github-policy-service bot added the Area-SettingsUI Anything specific to the SUI label Aug 21, 2024
@DHowett
Copy link
Member

DHowett commented Aug 21, 2024

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@DHowett DHowett enabled auto-merge (squash) August 21, 2024 22:08
@microsoft-github-policy-service microsoft-github-policy-service bot added Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. labels Aug 21, 2024
@DHowett DHowett merged commit ce92b18 into main Aug 21, 2024
@DHowett DHowett deleted the dev/pabhoj/sui_action_edit_fix branch August 21, 2024 22:43
DHowett pushed a commit that referenced this pull request Aug 21, 2024
Fixes a regression from the actions MVVM change in #14292 - attempting
to overwrite a keybinding was displaying a warning but propagating the
change before the user acknowledged it.

The overwrite key binding warning in the SUI works like before

Closes #17754

(cherry picked from commit ce92b18)
Service-Card-Id: PVTI_lADOAF3p4s4AmhmszgSF01Y
Service-Version: 1.21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-SettingsUI Anything specific to the SUI Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The "Overwrite key binding" warning is very broken

4 participants