-
Notifications
You must be signed in to change notification settings - Fork 23
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 setting near/far planes and listening for view update events #41
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, a minor nit about keeping a dirty bit in the clip info, since most of the structs in webxr-api
are intended to be immutable. Can we keep the mutable state somewhere else?
webxr-api/view.rs
Outdated
pub near: f32, | ||
pub far: f32, | ||
/// Was there an update that needs propagation to the client? | ||
update: bool, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, it would be nice if we didn't have to keep a dirty bit on the ClipPlanes
itself.
8feca87
to
3e6486e
Compare
@bors-servo r+ |
📌 Commit 3e6486e has been approved by |
Allow setting near/far planes and listening for view update events r? @asajeffrey I couldn't handle magicleap, sadly, since I can't get it to build, and it seems to manage its own projection matrices Fixes #32
|
💔 Test failed - checks-travis |
3e6486e
to
e77822c
Compare
@bors-servo r=asajeffrey |
📌 Commit e77822c has been approved by |
Allow setting near/far planes and listening for view update events r? @asajeffrey I couldn't handle magicleap, sadly, since I can't get it to build, and it seems to manage its own projection matrices Fixes #32
☀️ Test successful - checks-travis |
Allow setting near/far clip planes needs servo/webxr#41 r? @asajeffrey <!-- 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/24018) <!-- Reviewable:end -->
Allow setting near/far clip planes needs servo/webxr#41 r? @asajeffrey <!-- 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/24018) <!-- Reviewable:end -->
r? @asajeffrey
I couldn't handle magicleap, sadly, since I can't get it to build, and it seems to manage its own projection matrices
Fixes #32