Skip to content

Revert "SetZoom update the method to calculate the trans"#514

Merged
hannesa2 merged 4 commits intomasterfrom
zoom
Jan 23, 2023
Merged

Revert "SetZoom update the method to calculate the trans"#514
hannesa2 merged 4 commits intomasterfrom
zoom

Conversation

@hannesa2
Copy link
Collaborator

This reverts commit bc7b506.

@hannesa2
Copy link
Collaborator Author

close #507

@hannesa2
Copy link
Collaborator Author

hannesa2 commented Jan 22, 2023

As expected the zoom is now different (thanks to #513 the CI do a verification)
image

@SimFG it reverts your pull request #329 and handles #507 and #506
To be honest, I don't know if I should revert it or not. What do you think @SimFG and @bartek-wesolowski ?

I'm open to any solution.

@hannesa2
Copy link
Collaborator Author

close #506

@hannesa2
Copy link
Collaborator Author

And it founds more differences https://github.com/MikeOrtiz/TouchImageView/suites/10513010901/artifacts/522001897
I think the screenshot comparison mechanism is not perfect and the init status is confusing too
image

@bartek-wesolowski
Copy link

I think reverting the commit is the right thing to do. The current formula is incorrect. If we want to block panning of smaller images if their containing view is bigger then we should probably adjust getFixTrans() instead of setZoom().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants