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.

Sign up for GitHub

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

Jump to bottom

[charts] Improve interaction for mobile user #13109

Closed
alexfauquette opened this issue May 13, 2024 · 2 comments · Fixed by #13692
Closed

[charts] Improve interaction for mobile user #13109

alexfauquette opened this issue May 13, 2024 · 2 comments · Fixed by #13692
Assignees
@JCQuintas
Labels
component: charts This is the name of the generic UI component, not the React module! enhancement This is not a bug, nor a new feature

Comments

@alexfauquette
Copy link
Member

alexfauquette commented May 13, 2024 •
edited by github-actions bot
Loading

The PR #13043 added the support of axis interaction for touch interaction. But still have some room for improvement.

  1. Supporting touch move for the item interaction. This requires quite a big modification in the way item interactions are handled, and to make sure we do not deteriorate the performances
  2. Moving the tooltip to the top of the interaction point. Otherwise the tooltip is hidden by the thumb on mobile

Search keywords:
@alexfauquette alexfauquette added enhancement This is not a bug, nor a new feature component: charts This is the name of the generic UI component, not the React module! labels May 13, 2024
@JCQuintas
Copy link
Member

One path we could test without having to fully re-write the current behaviour is to use pointer events instead of the regular mouse/touch events.

@JCQuintas JCQuintas self-assigned this Jun 28, 2024
@JCQuintas JCQuintas mentioned this issue Jul 1, 2024
Merged
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 2, 2024

⚠️ This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

@alexfauquette: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.

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

@JCQuintas JCQuintas

Labels
component: charts This is the name of the generic UI component, not the React module! enhancement This is not a bug, nor a new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[charts] Improve charts interaction for mobile users JCQuintas/mui-x
2 participants
@JCQuintas @alexfauquette