mirror of
https://github.com/stenzek/duckstation.git
synced 2026-02-04 05:04:33 +00:00
[PR #549] Qt: Simplify Rebind All behavior #22
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Original Pull Request: https://github.com/stenzek/duckstation/pull/549
State: closed
Merged: Yes
The current behavior for Rebind All is to unconditionally rebind all buttons regardless of what the user selects in the question message box, with the selection only determining whether or not bindings are cleared before starting the full rebinding process. If Rebind All was pressed unintentionally, it seems that the user has to wait through the timeout process to prevent bindings from being changed.
To make this less prone to user error and more straightforward, Rebind All should first ask for user confirmation (like Clear All does), and if the user confirms, clear all bindings and run the rebinding.