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

Can't zoom the image on mobile device. #64

Closed
shikelong opened this issue Mar 23, 2017 · 13 comments
Closed

Can't zoom the image on mobile device. #64

shikelong opened this issue Mar 23, 2017 · 13 comments

Comments

@shikelong
Copy link

shikelong commented Mar 23, 2017

In my mobile chrome. if I try to zoom the image itself, (not the crop box). The picture will be lost。

But cropperjs's demo is ok.

after zoom image use fingers:

2017-03-23_162023

Regards.

@shikelong shikelong changed the title Can't scale the image on mobile device. Can't zoom the image on mobile device. Mar 23, 2017
@danazkari
Copy link

danazkari commented Mar 30, 2017

IDK if it's the same issue but I'm trying to zoom on mobile and I'm getting the next warning on the demo:
image
And it seems like the addEventListener implementation changed a bit https://developers.google.com/web/updates/2017/01/scrolling-intervention

FYI: I'm testing on a OnePlus 3T, Chrome 56, Android 7.0.

This issue is also happening on the app where we implemented this project, same issue, which is good! I think... 😛

@matthew-dev
Copy link

+1
I get exactly the same issue trying to zoom image on touchable laptop screen with Google Chrome v.56 and Opera v.43... in Firefox v.52 zooming works correctly

@SagiMedina
Copy link

Any solution available?

@Autushka
Copy link

Autushka commented Jun 6, 2017

Same problem here - zooming crashes the copper in Chrome on Android. Apparently, there was a similar problem with cropperjs that was fixed with the 3.x version.
https://github.com/fengyuanchen/cropper/issues/827

Version of cropperjs that is used in the master of the react-cropper at the moment:
"cropperjs": "^0.8.0"

And the latest version of cropperjs at the moment is:
1.0.0-rc.2

Quick Update:
Workaround the problem for me was bumping the dependency on "cropperjs": "^0.8.0" to version
1.0.0-rc.2

@Rauttis
Copy link

Rauttis commented Jul 18, 2017

Ran into the same issue, trying to zoom on mobile Chrome shifts the image in a weird way breaking the cropper.

@belief-cyf
Copy link
Contributor

@shikelong 你最后找到解决方案了嘛? 我也是碰到这个问题 在平板上的chrome浏览器内方法图片会崩溃 。

@belief-cyf
Copy link
Contributor

经过测试 chrome浏览器V56及以上的版本都会出现此问题

@shikelong
Copy link
Author

@belief-cyf Finally I disable zoom feature in mobile. I can't find a solution in the case.

@belief-cyf
Copy link
Contributor

I need this zoom function i cant disable~~

@matthew-dev
Copy link

matthew-dev commented Jul 31, 2017

I needed it aprox. 4 months ago, so as no provided solution I simply added buttons to provide zoom in and zoom out functions (and also disabled zoom on touch function)... It looks like this...

cropper image

@belief-cyf
Copy link
Contributor

fengyuanchen/cropperjs#161

cropperjs V1.0.0-RC resolve this err

Hope the author can go under the updated version

@belief-cyf
Copy link
Contributor

@matthew-dev @shikelong
I made some changes
update the cropper
You can use this temporarily, waiting for author update this project
https://github.com/belief-cyf/react-cropper

@shikelong
Copy link
Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants