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

Add microphone toggle button #1214

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ricnava00
Copy link

This adds a new button to toggle the microphone input. The main use is to quickly toggle the software blow on and off for games that require it (but it also disables other sources, so if the mic is set to real input it will also get disabled, and if it's set to disabled it will stay always disabled)
I had a quick look at the code while implementing this, so I don't know if it's the most effective way
I don't think I'll work more on this since it does what I needed, but it could be modified from a toggle to be a normal button, which blows when it's pressed
Also, I give up on git. I tried to push on the submodule and it may have made a mess. Let me know if I also need to create a pull request from the submodule, or just feel free to copy/paste the code (rafaelvcaetano/melonDS-android-lib@1175c71)

@christopher3242
Copy link

Also no mic

Screenshot_2023-11-29-08-01-33-462_me.magnum.melonds.jpg

I have attached

# Conflicts:
#	app/src/main/cpp/MelonDSAndroidJNI.cpp
#	app/src/main/java/me/magnum/melonds/MelonEmulator.kt
#	melonDS-android-lib
@ricnava00
Copy link
Author

I merged the new updates and I created a pull request on the lib repo rafaelvcaetano/melonDS-android-lib#5
I tried rebuilding everything and can confirm that the mic button appears for me and works as expected (from a fresh install, it is present in the default input layout; otherwise, it can be added normally)

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

Successfully merging this pull request may close these issues.

None yet

3 participants