Skip to content

[charts] Fix tooltip mark unexpected wrapping in mobile #19122

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
Merged
merged 1 commit into from
Aug 12, 2025
Merged

[charts] Fix tooltip mark unexpected wrapping in mobile #19122

merged 1 commit into from
Aug 12, 2025

Conversation

bernardobelchior
Copy link
Member

@bernardobelchior bernardobelchior commented Aug 11, 2025 •
edited
Loading

Before:

Screen.Recording.2025-08-11.at.11.06.47.mov

After:

Screen.Recording.2025-08-11.at.11.07.14.mov

@bernardobelchior bernardobelchior added type: bug It doesn't behave as expected. scope: charts Changes related to the charts. labels Aug 11, 2025
@mui-bot
Copy link

mui-bot commented Aug 11, 2025 •
edited
Loading

Deploy preview: https://deploy-preview-19122--material-ui-x.netlify.app/

Bundle size report

Bundle Parsed Size Gzip Size
@mui/x-data-grid 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-pro 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) 0B(0.00%)
@mui/x-charts 🔺+20B(+0.01%) 🔺+1B(0.00%)
@mui/x-charts-pro 🔺+20B(+0.01%) 🔺+1B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 0B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against ca4f5bf

@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Aug 11, 2025 •
edited
Loading

CodSpeed Performance Report

Merging #19122 will improve performances by 8.8%

Comparing bernardobelchior:fix-tooltip-mobile (ca4f5bf) with master (44aeba4)1

Summary

⚡ 1 improvements
✅ 9 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
⚡ ScatterChartPro with big data amount 437.1 ms 401.8 ms +8.8%

Footnotes

  1. No successful run was found on master (fac07f4) during the generation of this report, so 44aeba4 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

@bernardobelchior bernardobelchior marked this pull request as ready for review August 11, 2025 09:09
@bernardobelchior bernardobelchior merged commit 2a63728 into mui:master Aug 12, 2025
21 checks passed
@bernardobelchior bernardobelchior deleted the fix-tooltip-mobile branch August 12, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexfauquette alexfauquette alexfauquette approved these changes

@JCQuintas JCQuintas Awaiting requested review from JCQuintas JCQuintas is a code owner

Assignees
No one assigned
Labels
scope: charts Changes related to the charts. type: bug It doesn't behave as expected.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bernardobelchior @mui-bot @alexfauquette