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

webvr: touchstart is not sufficient as a user gesture #38

Closed
wants to merge 1 commit into
base: master
from

Conversation

2 participants
@Squareys
Contributor

Squareys commented Mar 13, 2018

Hi @mosra !

I finally solved the issue of the webvr example on mobile devices.

Also make sure to enable webvr support in your browser (docs are updated). Even on mobile devices this is required, except if a API token from google has been aquired for your origin (e.g. magnum.graphics, see https://docs.google.com/forms/d/e/1FAIpQLSfO0_ptFl8r8G0UFhT0xhV17eabG-erUWBDiKSRDTqEZ_9ULQ/viewform)

Cheers, Jonathan.

webvr: touchstart is not sufficient as a user gesture
Use touchend instead.

Signed-off-by: Squareys <squareys@googlemail.com>
@mosra

This comment has been minimized.

Owner

mosra commented Mar 14, 2018

Thanks!

Will merge it as soon as I can verify that it works on my phone (and will bother you again in case it doesn't, hehe).

The form says:

we will create a token for your site that will unlock these new features for the next 6 weeks.

I'm not sure if I want to renew this every 6 weeks, heh. Why is it controlled by a central authority like this?

@mosra mosra added this to TODO in Project management via automation Mar 14, 2018

@mosra mosra added this to the 2018.0b milestone Mar 26, 2018

@mosra

This comment has been minimized.

Owner

mosra commented Mar 26, 2018

Yaaas! I see a red cube! And a blue cube!

Thanks! 😍 🍺 Merged in bedfe74.

@mosra mosra closed this Mar 26, 2018

Project management automation moved this from TODO to Done Mar 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment