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

Potential issue when storing gamepad by index with 2 or more gamepads #35

Open
resir014 opened this issue Jan 8, 2021 · 1 comment
Open
Labels
bug Something isn't working needs investigation This issue needs more investigation

Comments

@resir014
Copy link
Owner

resir014 commented Jan 8, 2021

Description

There's a potential issue with how we currently store the gamepad index config. If the controller is plugged in out of order the overlay settings might be out of sync.

Current Settings

Any default settings with controllerIndex set to 0.

Additional Information

Reproduction steps:

  • Make sure both gamepads are plugged in, then on the customiser pick the first controller on the list.
  • Test if overlay works.
  • If works, unplug both controllers, then plug them back in, but plug the previously-used controller last.
  • Refresh the customiser, don't change any settings, then test the overlay.
@resir014 resir014 added the bug Something isn't working label Jan 8, 2021
@resir014
Copy link
Owner Author

After much consideration, the decision is made to remove the controller options for now, as there is no possible way to ensure that the OS will detect controllers in the right order after a restart.

We will change the gamepads context so that any controllers that are plugged in are added to the context value in an incrementing manner. This means only the most-recently connected controller will work with the overlay.

@resir014 resir014 added the needs investigation This issue needs more investigation label Oct 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs investigation This issue needs more investigation
Projects
None yet
Development

No branches or pull requests

1 participant