-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Comments
hi @Sophist-UK, what did you change the shortcut to? |
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. |
@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. |
Now they can reproduce and fix it with a confirmation ok button. |
@Sophist-UK is the issue resolved now? For improving UI, it is being tracked in #3445 |
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. |
i think #3445 is a better tracking issue. |
Environment
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.
The text was updated successfully, but these errors were encountered: