-
Notifications
You must be signed in to change notification settings - Fork 34
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
maxZoomFactor should apply on top of any zoom applied by the content scale #45
Comments
I can see why this can be confusing, but the explanation is that that image is already max zoomed in to fit its bounds. This can be verified by changing the zoom.mp4 |
I see, from a technical perspective this indeed correct. But from a user perspective it does feel like one image can be zoomed in more than others. Which was the feedback i was getting. Maybe a option where it starts the zoomlevel from the size it takes up? It is not really needed for me as the users just wanted a "limitless" zoom. So I have set to a rather big number. You can close this if you want. |
Agree, I'll think more about this.
This unfortunately has a downside that double tapping images in your app will zoom them all the way to 20x. |
I see, #32 could help with that |
Yeah, we would like to have something like |
Something that I have noticed is that zoomlevel seems very inconsistent. Sometimes I can barely zoom at all.
studio64_E1cOJMEQex.mp4
For now I have countered this by doing 20x.
see repro: https://github.com/MV-GH/ReproChannelCrash/tree/bug/inconsisent_zoom
The text was updated successfully, but these errors were encountered: