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

Resolved the zoom-in rotation issue #14

Open
wants to merge 1 commit into
base: with_navbar
Choose a base branch
from

Conversation

priyansh-narang2308
Copy link

Resolve #10
resolved the zoomin and rotation in the map
updated the zoomscript.js with two new fucntions getCurrentRotaion and applytransform()
and eventually put in the wheel and touchmove event listeners which solves the issue
Used the WebKitCSSMatrix which provides access to the underlying transformation matrix and the math function used..
This is the issue solved..
image

@priyansh-narang2308
Copy link
Author

@Abhinav-ark Kindly review the PR🫡🫡

@Abhinav-ark
Copy link
Owner

Abhinav-ark commented Mar 1, 2025

Screen.Recording.2025-03-01.235327.mp4

@priyansh-narang2308 I wanted the applied zoom also to exist, during rotation. This PR doesn't solve that problem. Moreover I noticed some rotation bugs at 270 degrees.

Screen.Recording.2025-03-02.000255.mp4

@Ashrockzzz2003 Ashrockzzz2003 added the AMWOC Part of ACM Amrita WOC S01 Contest. label Mar 1, 2025
@Ashrockzzz2003
Copy link
Collaborator

/bounty 40 @priyansh-narang2308

@priyansh-narang2308
Copy link
Author

Oh okk I see that means a glitch is happening @Abhinav-ark

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AMWOC Part of ACM Amrita WOC S01 Contest. AMWOC-accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Preserve rotation on zoom.
3 participants