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

PT Run Shortcut Key change does not save if window is closed before focus is lost #4276

Closed
Sophist-UK opened this issue Jun 12, 2020 · 8 comments
Labels
Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something Product-PowerToys Run Improved app launch PT Run (Win+R) Window

Comments

@Sophist-UK
Copy link

Sophist-UK commented Jun 12, 2020

Environment

Microsoft Windows [Version 10.0.18363.836]
PowerToys version: v0.18.2
PowerToys module for which you are reporting the bug (if applicable): Run

Steps to reproduce

Open Settings, select PT Run, change the Shortcut Key without clicking elsewhere and losing focus, close settings, reopen settings, select PT Run, see if change has persisted

Expected behavior

Change of Shortcut Key persists

Actual behavior

Change of Shortcut Key doesn't persist.

NOTE: Default Shortcut Key of Alt-Space is IMO not a good one as this is the shortcut key for the current active window Max/Min/Restore/Close popup.

@jyuwono
Copy link
Contributor

jyuwono commented Jun 12, 2020

hi @Sophist-UK, what did you change the shortcut to?

@jyuwono jyuwono added Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something Product-PowerToys Run Improved app launch PT Run (Win+R) Window labels Jun 12, 2020
@Sophist-UK
Copy link
Author

Ctrl-\

@androidSwDev099
Copy link

Ctrl-\

After you change it to Ctrl-, try clicking somewhere outside the PT Run Open PowerToys Run box, but still within the PowerToys Settings window so you can see it's Ctrl-\ before closing the window.

@Sophist-UK
Copy link
Author

@androidSwDev099 Yes - you seem to be correct. The new value is only being saved when the control loses focus.

Not great UI design when that is not obvious.

@androidSwDev099
Copy link

Now they can reproduce and fix it with a confirmation ok button.

@jyuwono
Copy link
Contributor

jyuwono commented Jun 12, 2020

@Sophist-UK is the issue resolved now? For improving UI, it is being tracked in #3445

@Sophist-UK
Copy link
Author

Or save the value whenever it changes not just when the control loses focus.

If there are other hotkey settings fields, this may be an issue common to all of them.

@Sophist-UK Sophist-UK changed the title PT Run Shortcut Key change does not save sometimes PT Run Shortcut Key change does not save if window is closed before focus is lost Jun 12, 2020
@crutkas
Copy link
Member

crutkas commented Jun 19, 2020

i think #3445 is a better tracking issue.

@crutkas crutkas closed this as completed Jun 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something Product-PowerToys Run Improved app launch PT Run (Win+R) Window
Projects
None yet
Development

No branches or pull requests

4 participants