Skip to content

[charts] Prevent overflow on charts tooltip #19123

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] Prevent overflow on charts tooltip #19123

merged 1 commit into from
Aug 12, 2025

Conversation

bernardobelchior
Copy link
Member

Before:

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

After:

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

@mui-bot
Copy link

mui-bot commented Aug 11, 2025 •
edited
Loading

Deploy preview: https://deploy-preview-19123--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%) 🔺+2B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) 🔺+2B(0.00%)
@mui/x-charts 🔺+46B(+0.01%) 🔺+20B(+0.02%)
@mui/x-charts-pro 🔺+46B(+0.01%) 🔺+22B(+0.02%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 🔺+5B(+0.01%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 🔺+1B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against 73220ee

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

codspeed-hq bot commented Aug 11, 2025

CodSpeed Performance Report

Merging #19123 will not alter performance

Comparing bernardobelchior:charts-tooltip-prevent-overflow (73220ee) with master (7db0f6f)1

Summary

✅ 10 untouched benchmarks

Footnotes

  1. No successful run was found on master (8270d2c) during the generation of this report, so 7db0f6f 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 14:31
@bernardobelchior bernardobelchior merged commit 060778c into mui:master Aug 12, 2025
24 of 25 checks passed
@bernardobelchior bernardobelchior deleted the charts-tooltip-prevent-overflow 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

@JCQuintas JCQuintas JCQuintas approved these changes

@alexfauquette alexfauquette Awaiting requested review from alexfauquette alexfauquette 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 @JCQuintas