pokerogue icon indicating copy to clipboard operation
pokerogue copied to clipboard

Swap A and B option doesn't work with the keyboard, only the gamepad.

Open Jaizu opened this issue 9 months ago • 7 comments

Tested in the following scenarios:

  • [ ] Options Menu
  • [ ] Main menu
  • [ ] Starter choose screen
  • [ ] Battle

chrome_ibjABsM8se

Edit: Is is possible that it only affects game controllers and not they keyboard binding? If so, I should rename my issue.

Jaizu avatar May 05 '24 10:05 Jaizu

Indeed, it's only for the gamepad, and in my case it works very well.

Kiriox94 avatar May 05 '24 10:05 Kiriox94

It should affect both imo ^^

Jaizu avatar May 05 '24 11:05 Jaizu

This should wait for #429

Jaizu avatar May 05 '24 11:05 Jaizu

If we make it affect both, I'd like to make it default to on.

Flashfyre avatar May 05 '24 14:05 Flashfyre

When I added the setting, it was never intended to affect both. This isn't a bug but rather a new feature suggestion. That's why the setting was named Swap A and B as the keyboard keys A and B do not correspond to Action and Cancel.

bennybroseph avatar May 05 '24 18:05 bennybroseph

Final call: no swap for keyboard, this issue can be closed.

Greenlamp2 avatar May 06 '24 08:05 Greenlamp2

this behavior is very confusing, especially since the Accept/Cancel mapping is the opposite of what retroarch does by default.

lolbinarycat avatar May 11 '24 21:05 lolbinarycat

No, this is not completed, only gamepad is supported to swap A/B button, keyboard should have the option in some form too.

Jaizu avatar May 12 '24 11:05 Jaizu

That's intentional, you'd need to make a new suggestion post. This option functions properly and is not bugged.

bennybroseph avatar May 12 '24 11:05 bennybroseph

i'm working on the full remaping of the gamepads, next will be the keyboard, don't worry, i've heard you https://github.com/pagefaultgames/pokerogue/pull/685

Greenlamp2 avatar May 12 '24 11:05 Greenlamp2

perhaps the options should also say "(only affects gamepad)" so people don't keep getting confused?

lolbinarycat avatar May 12 '24 18:05 lolbinarycat