Skip to content

feat: Enable touch-based panning and pinch-to-zoom in SchematicViewer on mobile devices #89

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

mohan-bee
Copy link

Closes: #87

/claim #87

Screen.Recording.2025-05-02.at.10.54.28.PM.mov

Copy link

vercel bot commented May 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
schematic-viewer ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 2, 2025 6:45pm

@imrishabh18
Copy link
Member

I tired it in mobile, can't zoom in!!

@imrishabh18
Copy link
Member

Make sure to actually test in a mobile before requesting reviews

@mohan-bee
Copy link
Author

@imrishabh18 OK !!

@mohan-bee
Copy link
Author

@imrishabh18 Ready for review !!

@mohan-bee mohan-bee changed the title Enable touch-based panning in SchematicViewer on mobile devices Enable touch-based panning and zooming in SchematicViewer on mobile devices May 2, 2025
@mohan-bee mohan-bee changed the title Enable touch-based panning and zooming in SchematicViewer on mobile devices feat: Enable touch-based panning and pinch-to-zoom in SchematicViewer on mobile devices May 2, 2025
Copy link
Member

@Anshgrover23 Anshgrover23 left a comment

Choose a reason for hiding this comment

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

@mohan-bee try in your android/IOS device not in browser android view and attached good video reference.

@mohan-bee
Copy link
Author

@Anshgrover23 Here is the tested mobile version!

Record_2025-05-03-11-34-31.mp4

@mohan-bee mohan-bee requested a review from Anshgrover23 May 3, 2025 06:13
Copy link
Member

@Anshgrover23 Anshgrover23 left a comment

Choose a reason for hiding this comment

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

@imrishabh18 passing to you.

@seveibar
Copy link
Contributor

seveibar commented May 5, 2025

@imrishabh18 afaik he just needs to update use mouse matrix transform, the diff for this looks a bit wack to me

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.

Click to interact doesnt work on mobile
4 participants