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

Touch input – Improve documentation: touch-action="none" needed #5674

Closed
thomasaull opened this issue Dec 22, 2018 · 3 comments
Closed

Touch input – Improve documentation: touch-action="none" needed #5674

thomasaull opened this issue Dec 22, 2018 · 3 comments
Assignees
Milestone

Comments

@thomasaull
Copy link

@thomasaull thomasaull commented Dec 22, 2018

I just spent an hour trying to figure out why my ArcRotateCamera was not working with touch input on iOS. Even the official playground example: https://www.babylonjs-playground.com/#12WBC#69 did work as long as I used the playground. Downloading the .zip and using the generated index.html broke the functionality.

Eventually I figured out the canvas needs to have the attribute touch-action="none" to make it work. Since I couldn't find it anywhere in the documentation, please consider adding it to save other users from pulling their hair out and wasting time :) Thanks!

@sebavan
Copy link
Member

@sebavan sebavan commented Dec 22, 2018

@deltakosh Should I do it in code instead ? it has been several questions and between css and attributes depending on the browser it is always a bit tricky.

Are there any good reason to not do it by code ?

@sebavan
Copy link
Member

@sebavan sebavan commented Dec 22, 2018

@thomasaull yup you are not the first one with the issue, we ll definitely do smthg about it.

@deltakosh deltakosh self-assigned this Dec 25, 2018
@deltakosh deltakosh added this to the 4.0 milestone Dec 25, 2018
@sebavan
Copy link
Member

@sebavan sebavan commented Dec 26, 2018

This will now be done automatically

@sebavan sebavan closed this Dec 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants