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

Allow generic-hand #121

Merged
merged 1 commit into from
Feb 7, 2024
Merged

Conversation

Manishearth
Copy link
Contributor

@Manishearth Manishearth commented Feb 6, 2024

It's unclear to me if we shoudl require "generic-hand" at all times or allow "generic-hand-select" input methods to omit "generic-hand" (thoughts @cabanier?)

Fixes immersive-web/webxr#1358


Preview | Diff

@toji
Copy link
Member

toji commented Feb 7, 2024

Ada mentioned on the issue:

It's odd because generic-hand is already included in the WebXR input profiles repo.

As such, this feels appropriate. LGTM

@Manishearth Manishearth merged commit 16b0371 into immersive-web:main Feb 7, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Feb 7, 2024
SHA: 16b0371
Reason: push, by Manishearth

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@Manishearth
Copy link
Contributor Author

Ah, sorry! Will do when back at computer.

@cabanier
Copy link
Member

cabanier commented Feb 7, 2024

Ah, sorry! Will do when back at computer.

yes, your change is not compatible with what we are currently doing.

@Manishearth
Copy link
Contributor Author

Reverted: #122

@Manishearth
Copy link
Contributor Author

#123 is the new PR

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.

Detecting if an XRInputSource is an auxiliary or a primary input source
3 participants