Skip to content

input: fix some binding regressions after strong typing rework#359

Merged
Supreeeme merged 4 commits into
Supreeeme:mainfrom
ImSapphire:sapphire/fix/oculus-touch-squeeze-trigger-click
Jun 12, 2026
Merged

input: fix some binding regressions after strong typing rework#359
Supreeeme merged 4 commits into
Supreeeme:mainfrom
ImSapphire:sapphire/fix/oculus-touch-squeeze-trigger-click

Conversation

@ImSapphire

Copy link
Copy Markdown
Contributor

Closes #358

@ImSapphire ImSapphire changed the title input: fix binds to grip/click and trigger/click on Oculus Touch input: fix some binding regressions after strong typing rework May 18, 2026
@ImSapphire ImSapphire force-pushed the sapphire/fix/oculus-touch-squeeze-trigger-click branch from 2dcf43c to 8134b29 Compare May 18, 2026 03:17
Comment thread src/input/profiles/oculus_touch.rs
Comment thread src/input/profiles.rs Outdated
"x" => Some(Self::X),
"y" => Some(Self::Y),
"application_menu" => Some(Self::Menu),
"application_menu" | "system" => Some(Self::Menu),

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not positive "system" should map to the menu button.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, on SteamVR 'system' does mean the menu button in some cases. I'm guessing it should be mapped to Self::System and then translate it to Menu for specific controllers where necessary?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I'll just drop that part and leave it to a follow-up to implement, having a hard time focusing right now..

Comment thread src/input/profiles.rs
@ImSapphire ImSapphire force-pushed the sapphire/fix/oculus-touch-squeeze-trigger-click branch from 015cb30 to a59a2da Compare May 30, 2026 01:58
@ImSapphire ImSapphire requested a review from Supreeeme May 30, 2026 01:58
@ImSapphire ImSapphire force-pushed the sapphire/fix/oculus-touch-squeeze-trigger-click branch from a59a2da to f2ae8ad Compare June 5, 2026 20:14
@Supreeeme Supreeeme merged commit 3131956 into Supreeeme:main Jun 12, 2026
3 checks passed
@ImSapphire ImSapphire deleted the sapphire/fix/oculus-touch-squeeze-trigger-click branch June 14, 2026 08:02
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.

[Regression] Input bindings rejected for oculus/touch_controller profile since strong types refactor (Resonite)

3 participants