Skip to content

Conversation

alexfauquette
Copy link
Member

The issue is that previousTextLimit was updated even is their is no text to display. So frequently for log scale axis, the 90k tick which is very close to the 100k tick update the previousTextLimit and prevent the 100k tick to render.

Before/after

image image

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

mui-bot commented Sep 12, 2025

Deploy preview: https://deploy-preview-19547--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%) ▼-1B(0.00%)
@mui/x-charts 🔺+42B(+0.01%) 🔺+11B(+0.01%)
@mui/x-charts-pro 🔺+42B(+0.01%) 🔺+10B(+0.01%)
@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 7231bc4

Copy link

codspeed-hq bot commented Sep 12, 2025

CodSpeed Performance Report

Merging #19547 will improve performances by 8.45%

Comparing alexfauquette:fix-tex-overlap (7231bc4) with master (980daf0)1

Summary

⚡ 1 improvement
✅ 9 untouched

Benchmarks breakdown

Benchmark BASE HEAD Change
ScatterChartPro with big data amount 437.1 ms 403.1 ms +8.45%

Footnotes

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

Copy link
Member

@bernardobelchior bernardobelchior left a comment

Choose a reason for hiding this comment

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

Nice catch 👍

@alexfauquette alexfauquette merged commit 844f100 into mui:master Sep 12, 2025
22 checks passed
JCQuintas pushed a commit to JCQuintas/mui-x that referenced this pull request Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: charts Changes related to the charts. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants