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

[charts][zoom] Support multi touch zoom (mostly for mobile) #16660

Open
oliviertassinari opened this issue Feb 20, 2025 · 4 comments
Open

[charts][zoom] Support multi touch zoom (mostly for mobile) #16660

oliviertassinari opened this issue Feb 20, 2025 · 4 comments
Labels
design This is about UI or UX design, please involve a designer enhancement This is not a bug, nor a new feature mobile Targets mobile platform waiting for 👍 Waiting for upvotes

Comments

@oliviertassinari
Copy link
Member

oliviertassinari commented Feb 20, 2025

Summary

This issue is similar to #13798 in the sense that it's about improving the zoom feature, but this time not on desktop, on mobile.

I would expect to be able to zoom on the chart like I can zoom on the canvas on Google Maps.

Examples

Expected (but no rotations):

Screen.Recording.2025-02-20.at.01.13.50.mov

Actual: https://next.mui.com/x/react-charts/zoom-and-pan/

Screen.Recording.2025-02-20.at.01.16.33.mov

Motivation

Improve UX

Search keywords: -

@oliviertassinari oliviertassinari added design This is about UI or UX design, please involve a designer enhancement This is not a bug, nor a new feature mobile Targets mobile platform status: waiting for maintainer These issues haven't been looked at yet by a maintainer waiting for 👍 Waiting for upvotes labels Feb 20, 2025
@michelengelen michelengelen removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Feb 20, 2025
@JCQuintas
Copy link
Member

@mui/xcharts this used to work in v7. Is it a regression?

@alexfauquette
Copy link
Member

this used to work in v7. Is it a regression?

Effectively. But it does not work on mui.com either so the regression also occurred on the v7

@JCQuintas
Copy link
Member

this used to work in v7. Is it a regression?

Effectively. But it does not work on mui.com either so the regression also occurred on the v7

Did you try on mobile. It needs to be multi-touch enabled.

@oliviertassinari
Copy link
Member Author

oliviertassinari commented Feb 21, 2025

I have never seen this one work. I assumed that it was a feature request.


Speaking of regression, fixing that one in: #16675. I might help a bit as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design This is about UI or UX design, please involve a designer enhancement This is not a bug, nor a new feature mobile Targets mobile platform waiting for 👍 Waiting for upvotes
Projects
None yet
Development

No branches or pull requests

4 participants