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 a pairing utility for Wiimotes to Cemu #941

Merged
merged 3 commits into from
Sep 6, 2023
Merged

Add a pairing utility for Wiimotes to Cemu #941

merged 3 commits into from
Sep 6, 2023

Conversation

yeah-its-gloria
Copy link
Contributor

@yeah-its-gloria yeah-its-gloria commented Aug 16, 2023

figured since Cemu can directly interact with Wiimotes this would be a handy tool to have :)

As of now, this is only available on Windows.
This also successfully pairs Wii U Pro Controllers, but since Cemu's
  Wiimote implementation doesn't support Classic Controllers, it doesn't
  fully work.
@Squall-Leonhart
Copy link
Contributor

you should aim to have feature parity on all supported platforms

@yeah-its-gloria
Copy link
Contributor Author

bluez does this already

@Exzap
Copy link
Member

Exzap commented Sep 2, 2023

Seems like you left debug logging in there (OutputDebugStringW). Can you clean that up?

src/gui/PairingDialog.cpp Outdated Show resolved Hide resolved
src/gui/PairingDialog.cpp Outdated Show resolved Hide resolved
@Fs00
Copy link
Contributor

Fs00 commented Sep 2, 2023

Just a thought: wouldn't it make more sense to open the pairing utility via a button in the input settings window, instead of having an entry in the Tools menu?

@yeah-its-gloria
Copy link
Contributor Author

yeah-its-gloria commented Sep 4, 2023

I've implemented all those fixes, thanks for letting me know

- Move Wiimote pairing to the input menu
- Make text translatable
- Remove debug leftovers
@Exzap Exzap merged commit d7f0d67 into cemu-project:main Sep 6, 2023
5 checks passed
@Exzap
Copy link
Member

Exzap commented Sep 6, 2023

thanks!

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.

4 participants