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

DolphinQt/Mapping: Add setting to enable waiting for alternate mappings. #13320

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jordan-woyak
Copy link
Member

@jordan-woyak jordan-woyak commented Feb 1, 2025

This feature was removed in #9685 because the delay it introduces during button mappings is annoying.

The default behavior is unchanged.
By enabling this new setting, auto generation of | mappings using sequential button presses is again possible.
image

Pressing X followed by Z with the setting enabled:

Screencast_20250201_001656.webm

Users occasionally ask how to map two buttons without having to press them at the same time.
Telling them to enable this setting is easier than explaining how to use the advanced mapping window.

I also did some other minor code cleanups.

Copy link
Member

@dreamsyntax dreamsyntax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only concern is 0 to 750ms wait, otherwise lgtm

@jordan-woyak
Copy link
Member Author

Only concern is 0 to 750ms wait, otherwise lgtm

The delay is only introduced when the setting is enabled. And the setting needs to be enabled every time the dialog is opened when it's wanted,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants