Skip to content
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

Fix x and y translation issue on double tap #149

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bastionkid
Copy link

@bastionkid bastionkid commented Aug 3, 2024

Fixed the mPosX & mPosY values so that double tap to zoom does not cause the actual pdf bitmap to go out of screen bounds.

After applying the fix

Kapture.2024-08-04.at.07.28.56.mp4

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for contributing to this project. This is your first pull request and we are so glad to have you onboard. We will review the request and get back to you soon. We love your contributions! Join our Discord community here to discuss this PR or ask questions.

@bastionkid bastionkid mentioned this pull request Aug 3, 2024
@VEZE
Copy link

VEZE commented Aug 3, 2024

Right side of screen taps are not accurate, don't think it is fixed.

@bastionkid
Copy link
Author

bastionkid commented Aug 4, 2024

Right side of screen taps are not accurate, don't think it is fixed.

@VEZE Good observation. That was happening because of redundant clamping that was being applied. It is fixed in this commit.

After clamping removal

Kapture.2024-08-04.at.07.28.56.mp4

@afreakyelf
Copy link
Owner

Hey, this looks great. Did you test the behavior when you scroll while zoomed in? Mind posting the video?

@afreakyelf afreakyelf self-requested a review August 7, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants