-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
(At least!) Pan Recognizer broken in Chrome 55 and Mobile Safari #1068
Comments
Do you have any hints as to why this is failing on touch devices @arschmitz @runspired ? |
just sanity checking -- you have an unclosed string in a couple of your console.info invocations. |
thanks @beeduul , I didn't really copy my code here, but rather wrote a "minimal" example and totally forgot to double check :) |
An example is not "minimal" if a heavy library like ember is included. So maybe the bug is in ember-gestures? |
@herom the other thing i notice in your code is that you are using camelCase for the pan event strings. http://hammerjs.github.io/recognizer-pan/ shows they're lowercase. |
I did my own implementation of drag&drop with the latest
ember-gestures
lib, which depends onhammer.js
and is also developed by @runspired. On desktop, all of the code works pretty well, but on Mobile/Chrome devtools emulator/Touch devices, this fails big time!Is there any fix for this? I'm working on a rather big project which decided to go with
hammer.js
andember-gestures
a few months ago as it seemed like the most promising/actively developed library out there... so this is quite frustrating...Here is a short code sample of my implementation (if it helps):
The text was updated successfully, but these errors were encountered: