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

Fix 8bitdo Neo Geo Shoulder buttons #942

Closed
wants to merge 1 commit into from

Conversation

GrumpyGopher
Copy link
Contributor

#933 Fix issue with 8bitdo NeoGeo pad shoulder buttons not detected. The 8bitdo Neo Geo Pad is most like the 8BitDo SNES pads with just L1 and R1 shoulder buttons. In version 1.8.4 this line didn't exist at all and it just defaulted to default 8bitdo settings. Compiled this locally and tested and it seemed to fix the problem.

Fix issue with 8bitdo NeoGeo pad shoulder buttons not detected.  The 8bitdo Neo Geo Pad is most like their SNES pads with just L1 and R1 shoulder buttons.
@darthcloud
Copy link
Owner

See #933 while this PR worked to get the trigger as L1/R1 my original objective was to have them on L2/R2 by default.

That is now working in latest beta.

Thanks for the PR in anycase!

@darthcloud darthcloud closed this Apr 9, 2024
@GrumpyGopher
Copy link
Contributor Author

Cool I'll give it a try sometime. A little curious why you wanted them defaulted to L2/R2 when they are digital buttons and they are even labeled L and R just like L1 and R1 on the 8bitdo SN30Pro+ controllers?

@darthcloud
Copy link
Owner

the main trigger are In BlueRetro are L2,R2 or LT, RT, or ZL,ZR the top one are the secondary. This keeps things consistent across all systems.

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.

2 participants