-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
[1.0.0][Chrome m79] Gyro not working - device orientation not taken into account #4355
Comments
Any errors on console? |
Unfortunately, nothing about that... A quick search for Going back to previous commits doesn't solve the issue. |
For comparison, https://threejs.org/examples/misc_controls_deviceorientation.html works in Chrome m79. |
If anyone wants to take a shot at it before I do, here are some starting points: |
Orientation used to be handled by webvr-polyfill but we don't use it anymore in Chrome m79? That's why it's not working in Chrome m79 but works fine in other browsers.
Fixing it on our side, already got a working build but we still need to do testing, PR will be submitted tomorrow. |
Magic window tracking is also using the Polyfill and only enabled when WebXR is not available. After Chrome m79, ARCore devices come with WebXR enabled so we have to make sure the Polyfill also kicks in that case. |
If you disable flag "WebXR Device API" m79 - then gyro is working |
It works if you disable WebXR because then it uses the polyfill. It's not a proper fix though as you just can't ask all end users to disable the WebXR API. It's also counter productive as the goal is to support the actual WebXR spec. |
… DeviceOrientationControls without the DeviceOrientationEvent.requestPermission logic that is handled by A-Frame (fix aframevr#4355) Co-authored-by: MK <maxence@lucidweb.io> Co-authored-by: Diego Marcos <diego.marcos@gmail.com>
Description:
Device orientation is not taken into account anymore in non-VR mode (aka magic window).
Works on Chrome m78, breaks in m79.
Tested on
Logging the
deviceorientation
event produces correct result.The text was updated successfully, but these errors were encountered: