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

Added latching buttons (Android) #12119

Merged
merged 1 commit into from Aug 28, 2023

Conversation

ThunderousEcho
Copy link
Contributor

Added latching buttons, buttons which turn on when you press them and off when you press them again.
This is necessary to play games which require you to press more than 2 buttons at once.

@JosJuice
Copy link
Member

This is something I think will make a bunch of users happy :)

I'm planning on adding a system where you can add custom buttons to the touch overlay that you then can map to whatever you want using Dolphin's input binding system. By then using the toggle function, you would be able to achieve essentially the same functionality, but... Well, that way of doing things is a lot less discoverable for users, so I think adding a latching system like this would be nice even if it technically will be duplicated functionality eventually.

Didn't find anything in the code that needed commenting on, but let's see if @t895 would also like to review this.

Copy link
Contributor

@t895 t895 left a comment

Choose a reason for hiding this comment

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

Just a couple codestyle nitpicks. Otherwise, LGTM

@JosJuice
Copy link
Member

Please squash the commits into one.

@JosJuice
Copy link
Member

Sorry, I meant all the commits.

Added latching buttons, buttons which turn on when you press them and off when you press them again.
@t895 t895 merged commit 153a954 into dolphin-emu:master Aug 28, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants