-
Notifications
You must be signed in to change notification settings - Fork 885
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
Resizer Touch events #466
Comments
I am also struggling with this issue. I came across a pr in which these lines of code were added to the croppie library, but it does not seem to be present in the latest released version. Any update as to which version of croppie is needed in order to support touch events? |
I abandoned this library for a different solution. However, it's easy to
fix.
בתאריך יום ו׳, 20 ביולי 2018, 18:18, מאת Amy Troppe <
notifications@github.com>:
… I am also struggling with this issue. I came across a pr in which these
lines of code were added to the croppie library, but it does not seem to be
present in the latest released version.
Any update as to which version of croppie is needed in order to support
touch events?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#466 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AgwLcdjgzX1AWc-Gk1KPZel32ht2iX8Eks5uIfS2gaJpZM4R7wGR>
.
|
I tried editing my local copy but it did not work. |
check out my PR This is an old version though. It's been a long time since I worked with it. Hope it helps. |
I too faced this problem. |
I downloaded an instance of croppie a week ago. The touch resize events didn't work. After inspection I found that the code was missing but here I see it.
Is this new?
` if (vr) {
vr.addEventListener('mousedown', mouseDown);
vr.addEventListener('touchstart', mouseDown);
}
The text was updated successfully, but these errors were encountered: