Bugfix Precropping issue

[margorski]

_reCenterImage function now takes into account child widget aspect
ratio when calculating boundaries for offset clamping.

[xclud]

Thank you for the great job.

I am facing one small issue where the cropped image is not pulled back into the view correctly. See the attachment please.

[margorski]

Yes. That is intended. I described that in github issue. For our purpose, the current behaviour is intended so that is how it works now. If you would like to have it work like before with limiting the pan to the inside of the picture then the calculation needs to be more complex.

[xclud]

It's good to have an option of disabling this behavior. But i think it's best to have it enabled by default. What do you think?

[xclud]

Animations also come handy to have as another option.

[margorski]

Yeah, both animation and that behaviour could be parametrized.
However calculating clamp of pan movement in a previous way may be a little more tricky. Still, I think I may find some time trying to implement that next week.

[xclud]

I appreciate your efforts.

[margorski]

Sorry @xclud but I cannot find the time right now to finish that changes as proposed. Maybe I will have more time to finish that in the new year, but right now changes won't happen. So if you have time to finish that goes on, if not then let's wait.

[duytq94]

If we disable rotation like this PR #66 and combine it with this Bugfix Precropping PR, then everything works perfectly.
These PRs will fix the issue and that is the general UX user need (like Instagram at change avatar feature, you can try it)

[eduardohr-muniz]

This problem has been solved, I use zoom, rotation, and the crop, in my system, but the crop if you slide it to the position you want to crop it comes back by itself, is it some configuration or is it really with the gub, and congratulations by packge!