Skip to content

[Shortcut Guide] Close SG window when win shortcut was pressed#11355

Merged
mykhailopylyp merged 4 commits into
microsoft:feature/shortcutguidefrom
mykhailopylyp:new-ux
May 19, 2021
Merged

[Shortcut Guide] Close SG window when win shortcut was pressed#11355
mykhailopylyp merged 4 commits into
microsoft:feature/shortcutguidefrom
mykhailopylyp:new-ux

Conversation

@mykhailopylyp

@mykhailopylyp mykhailopylyp commented May 19, 2021

Copy link
Copy Markdown
Contributor

Summary of the Pull Request

What is this about:

What is include in the PR:

  1. Close SG window if
  • Win key was pressed and released
  • Win+key is pressed(e.g Win+A, Win+Shift)
  • ESC was pressed. Moved it from the window procedure
  • The shortcut is pressed again
  1. Update ShortcutSession telemetry event with close type
  2. Fix crash when thumb_window is not initialized. To reproduce it start PT in debug mode, press Win+D and try to open SG window.

How does someone test / validate:
Try to close SG window with ESC key, pressing the shortcut again, pressing Win+key shortcut, pressing and releasing just win key

Quality Checklist

  • Linked issue: #xxx
  • Communication: I've discussed this with core contributors in the issue.
  • Tests: Added/updated and all pass
  • Installer: Added/updated and all pass
  • Localization: All end user facing strings can be localized
  • Docs: Added/ updated
  • Binaries: Any new files are added to WXS / YML

Contributor License Agreement (CLA)

A CLA must be signed. If not, go over here and sign the CLA.

@mykhailopylyp mykhailopylyp added the Product-Shortcut Guide Refers to the Shortcut Guide PowerToy label May 19, 2021
@mykhailopylyp mykhailopylyp requested a review from enricogior May 19, 2021 13:37
@enricogior

Copy link
Copy Markdown
Contributor

Tested, it's a very nice improvement to the SG experience!

@mykhailopylyp mykhailopylyp merged commit cd07430 into microsoft:feature/shortcutguide May 19, 2021
@mykhailopylyp mykhailopylyp deleted the new-ux branch May 19, 2021 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Product-Shortcut Guide Refers to the Shortcut Guide PowerToy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants