-
Notifications
You must be signed in to change notification settings - Fork 140
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
Swiping does not work on iOS 5 #20
Comments
Seems the issue happens because of two UIScrollView conflicting - script use one to establish scrolling between images and another to make zoom-in/zoom-out. On some images this construction fails. A simple workaround would be to modify EGOPhotoScrollView.h:
if (self.zoomScale > 1.0f) { |
Thanks man, works like a charm. |
This worked great here. The problem was really only happening on a very small subset of pictures. |
Thanks, works great! |
Thx! You save my project :) |
Awesome, agonych! Thumbs up for you. |
You might as well consider to use Three20 library (http://three20.info/) which is newer and have a photo gallery module. |
Ya, thanks, I know three20, but the I prefer EGOPhotoViewer since it has fewer dependencies. |
Well, EGOPhotoViewer seems to be no longer developed |
Thanks |
W0W! Thanks a bunch for this. What I don't understand is why this is an issue for only a few photos? |
Sweet! this helped me as well! sure wish the owner of the repo would patch this so in the future everyone gets this fix |
Thnx!!! |
Im currently testing my apps on iOS 5 and see a problem when using the EGOPhotoViewer in landscape mode.
The swiping between photos does not work anymore in landscape. All works fine in portrait.
Update 1: I looked in to it somewhat further and discovered that it only does not work if you try to swipe touching the photo's but if you use any black areas next to the photo it works fine.
Seems the scrollViewDidScroll events are nog coming when swiping the photo in landscape modus.
Update 2: If i disable the userInteraction of the EGOPhotoImageView it all works fine. But then you dont receive push events and cant hide / show the toolbars.
The text was updated successfully, but these errors were encountered: