Skip to content

Conversation

bernardobelchior
Copy link
Member

Add demo for log-scale ticks without labels.

Screen.Recording.2025-08-12.at.10.21.30.mov

@bernardobelchior bernardobelchior added docs Improvements or additions to the documentation. scope: charts Changes related to the charts. labels Aug 12, 2025
@@ -47,7 +47,6 @@ const salesData = [150, 120, 200, 180, 90, 160, 140, 110, 85];
const profitData = [45, 35, 80, 65, 25, 55, 50, 40, 30];

const chartConfig = {
width: 600,
Copy link
Member Author

Choose a reason for hiding this comment

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

This was breaking the page on mobile.

Before:

Image

After:

Image

@mui-bot
Copy link

mui-bot commented Aug 12, 2025

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

Updated pages:

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 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) 🔺+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 817d7e5

Copy link

codspeed-hq bot commented Aug 12, 2025

CodSpeed Performance Report

Merging #19152 will degrade performances by 8.32%

Comparing bernardobelchior:axis-tick-no-labels-demo (817d7e5) with master (2a63728)1

Summary

❌ 1 regressions
✅ 9 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark BASE HEAD Change
ScatterChartPro with big data amount 402 ms 438.5 ms -8.32%

Footnotes

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

@bernardobelchior bernardobelchior added the type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. label Aug 12, 2025
@@ -202,7 +202,7 @@ async function main(argv) {
}

tsxFiles.forEach((filePath) => {
fs.promises.watchFile(filePath, { interval: 500 }, async () => {
fs.watch(filePath, { interval: 500 }, async () => {
Copy link
Member

Choose a reason for hiding this comment

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

Is it creating issue?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes

@bernardobelchior bernardobelchior force-pushed the axis-tick-no-labels-demo branch from 2321432 to fcab790 Compare August 13, 2025 09:25
@bernardobelchior bernardobelchior force-pushed the axis-tick-no-labels-demo branch from fcab790 to 817d7e5 Compare August 14, 2025 07:38
@bernardobelchior bernardobelchior merged commit 3c27f53 into mui:master Aug 14, 2025
20 of 21 checks passed
@bernardobelchior bernardobelchior deleted the axis-tick-no-labels-demo branch August 14, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation. scope: charts Changes related to the charts. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants