maxZoomFactor should apply on top of any zoom applied by the content scale

[MV-GH]

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

[saket]

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 contentScale to ContentScale.Inside:

zoom.mp4

[MV-GH]

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.

[saket]

Agree, I'll think more about this.

It is not really needed for me as the users just wanted a "limitless" zoom. So I have set to a rather big number.

This unfortunately has a downside that double tapping images in your app will zoom them all the way to 20x.

[MV-GH]

I see, #32 could help with that

[FooIbar]

Yeah, we would like to have something like maxUserZoom instead of the current maxZoomFactor.