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

openxr: support left hands and grips #67

Merged
merged 6 commits into from Oct 9, 2019

Conversation

@Manishearth
Copy link
Member

Manishearth commented Oct 8, 2019

Servo PR incoming, can land independently

r? @jdm @asajeffrey

@Manishearth
Copy link
Member Author

Manishearth commented Oct 8, 2019

Consumed in servo/servo#24390. The per-frame click is not consumed, since I'm blocked on servo/servo#24389

webxr/openxr/input.rs Outdated Show resolved Hide resolved
@Manishearth Manishearth force-pushed the Manishearth:sinister-intents branch 2 times, most recently from 730c515 to 1057fa8 Oct 8, 2019
@Manishearth
Copy link
Member Author

Manishearth commented Oct 8, 2019

Tested and works

@Manishearth
Copy link
Member Author

Manishearth commented Oct 8, 2019

I recommend this is reviewed one commit at a time because it's mostly moving code around but that's not obvious in the diff view

@Manishearth Manishearth force-pushed the Manishearth:sinister-intents branch from adce44f to f35b4b3 Oct 8, 2019
@Manishearth Manishearth force-pushed the Manishearth:sinister-intents branch from 13e64b9 to 1c6907e Oct 9, 2019
@jdm
Copy link
Member

jdm commented Oct 9, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Oct 9, 2019

📌 Commit 1c6907e has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Oct 9, 2019

Testing commit 1c6907e with merge ec49fdd...

bors-servo added a commit that referenced this pull request Oct 9, 2019
openxr: support left hands and grips

Servo PR incoming, can land independently

r? @jdm @asajeffrey
@bors-servo
Copy link
Contributor

bors-servo commented Oct 9, 2019

☀️ Test successful - checks-travis
Approved by: jdm
Pushing ec49fdd to master...

@bors-servo bors-servo merged commit 1c6907e into servo:master Oct 9, 2019
3 checks passed
3 checks passed
Travis CI - Pull Request Build Passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@Manishearth Manishearth deleted the Manishearth:sinister-intents branch Oct 9, 2019
bors-servo added a commit to servo/servo that referenced this pull request Oct 9, 2019
Support grip spaces in WebXR

Requires servo/webxr#67

Uses the support added in servo/webxr#67 to expose an optional grip space.

The per-frame click added there can't yet be consumed because we need to hook up gamepads (see #24389)

r? @asajeffrey @jdm

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/24390)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.