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

[Feature] sessiongranted support for VRButton to enable in-VR navigation #23110

Merged
merged 1 commit into from
Dec 30, 2021

Conversation

hybridherbst
Copy link
Contributor

The sessiongranted event is the current proposal for immersive web navigation (https://github.com/immersive-web/navigation#api-proposal). While still a proposal and only working in the Oculus Quest browser right now, it allows building connected VR pages (and AR pages, but that's not part of this PR).

Description

Navigation between pages was causing the session to exit and the user having to initiate the session again. If already authorized, sessiongranted allows to keep the VR session running instead.

A-Frame has implemented this a while ago already (https://aframe.io/docs/1.2.0/components/link.html) and people are creating nice demos with it, now it's coming to three.js :)

Notes

Currently, session navigation needs to be enabled via chrome://flags/#webxr-navigation-permission in the Oculus browser.

@mrdoob mrdoob added this to the r137 milestone Dec 29, 2021
@mrdoob
Copy link
Owner

mrdoob commented Dec 29, 2021

@cabanier Do you know if this will stay behind a flag for much longer?

@cabanier
Copy link
Contributor

I'm going to work with @toji on defining a new spec for immersive navigation. Until that is done and implemented, we will keep this behind a flag.

@mrdoob
Copy link
Owner

mrdoob commented Dec 29, 2021

Should I not merge this then?

@cabanier
Copy link
Contributor

Should I not merge this then?

I think it's OK to merge. The holdup is more about providing a trusted UI. The sessiongranted event is not likely going to change.

@hybridherbst
Copy link
Contributor Author

@cabanier is there a place (new issue in the /navigation repo?) to discuss ideas for that trusted UI? Got some input as well.

@cabanier
Copy link
Contributor

@cabanier is there a place (new issue in the /navigation repo?) to discuss ideas for that trusted UI? Got some input as well.

immersive-web/navigation#3

@mrdoob mrdoob merged commit ac30cac into mrdoob:dev Dec 30, 2021
@mrdoob
Copy link
Owner

mrdoob commented Dec 30, 2021

Thanks!

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.

3 participants