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

[ Control Configuration ] - The side mouse buttons cannot be bound to an action #187

Closed
DAK404 opened this issue Apr 26, 2024 · 7 comments · Fixed by #291
Closed

[ Control Configuration ] - The side mouse buttons cannot be bound to an action #187

DAK404 opened this issue Apr 26, 2024 · 7 comments · Fixed by #291
Labels
enhancement Something to modernize things request Request for a feature

Comments

@DAK404
Copy link

DAK404 commented Apr 26, 2024

Summary: The side mouse buttons cannot be bound to an action (Switch primary, switch secondary, etc.)

Platform: Windows 11 64 Bit
Architecture: x86_64

Steps to check:

  • Open Game
  • Choose Pilot
  • Options -> Config -> Mouse
  • Try to bind the mouse side button to an action (does not work as of now: 26-April-2024)

Note:
While creating a new pilot, I chose default keyboard option.

Please let me know if there are any log files or additional data is required to be captured and uploaded.

@winterheart
Copy link
Collaborator

We still using DirectX DInput 6.0 API for Windows build, no wonder there no additional mouse buttons for binding...

@Lgt2x
Copy link
Collaborator

Lgt2x commented Apr 26, 2024

Let's keep the issue open to solve that when we migrate framework!

@Lgt2x Lgt2x added the request Request for a feature label Apr 26, 2024
@JeodC JeodC added the enhancement Something to modernize things label May 2, 2024
@DAK404
Copy link
Author

DAK404 commented May 2, 2024

Piccu Engine v1.0.1 supports mouse side buttons!

@JeodC
Copy link
Member

JeodC commented May 2, 2024

Piccu Engine v1.0.1 supports mouse side buttons!

That's cool, but unfortunately we can't bring it over just yet because of pending sdl changes, which may make it incompatible. We'll visit this when we're ready, don't worry. :)

@DAK404
Copy link
Author

DAK404 commented May 2, 2024

Yep yep. Just documenting the available info 😄

@icculus
Copy link
Contributor

icculus commented May 6, 2024

That's cool, but unfortunately we can't bring it over just yet because of pending sdl changes, which may make it incompatible. We'll visit this when we're ready, don't worry. :)

Were Piccu Engine's fixes for this only in the Windows-specific code? You should get them in there, even if we end up moving Windows to SDL2 as well, as that might take a little time.

@JeodC
Copy link
Member

JeodC commented May 6, 2024

That's cool, but unfortunately we can't bring it over just yet because of pending sdl changes, which may make it incompatible. We'll visit this when we're ready, don't worry. :)

Were Piccu Engine's fixes for this only in the Windows-specific code? You should get them in there, even if we end up moving Windows to SDL2 as well, as that might take a little time.

Good catch, I didn't look at it close enough. Thanks!

@JeodC JeodC closed this as completed in #291 May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Something to modernize things request Request for a feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants