Skip to content

[charts] Fix bug when min is in axis domain#22827

Merged
alexfauquette merged 1 commit into
mui:masterfrom
alexfauquette:fix-min-max
Jun 16, 2026
Merged

[charts] Fix bug when min is in axis domain#22827
alexfauquette merged 1 commit into
mui:masterfrom
alexfauquette:fix-min-max

Conversation

@alexfauquette

Copy link
Copy Markdown
Member

Closes #22822

@alexfauquette
alexfauquette requested a review from JCQuintas as a code owner June 16, 2026 09:14
@alexfauquette alexfauquette added type: bug It doesn't behave as expected. scope: charts Changes related to the charts. labels Jun 16, 2026
@code-infra-dashboard

code-infra-dashboard Bot commented Jun 16, 2026

Copy link
Copy Markdown

Deploy preview

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

Bundle size

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%) 0B(0.00%)
@mui/x-charts-premium 0B(0.00%) 0B(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%)
@mui/x-license 0B(0.00%) 0B(0.00%)

Details of bundle changes

Performance

Total duration: 1,393.24 ms ▼-686.72 ms(-33.0%) | Renders: 67 (+0)

Test Duration Renders
Heatmap: 100x100 grid 448.57 ms ▼-224.17 ms(-33.3%) 2 (+0)
LineChart stacked area with multiple series 120.77 ms ▼-86.82 ms(-41.8%) 2 (+0)
SparkLineChart grid of 100 instances 114.38 ms ▼-60.18 ms(-34.5%) 1 (+0)
CandlestickChart with big data amount (webgl renderer) 143.19 ms ▼-43.64 ms(-23.4%) 5 (+0)
RangeBarChart with big data amount 55.18 ms ▼-31.27 ms(-36.2%) 3 (+0)

…and 19 more (+2 within noise) — details


Check out the code infra dashboard for more information about this PR.

@JCQuintas

Copy link
Copy Markdown
Member

probably a visual test would be easier to keep track of

@alexfauquette
alexfauquette merged commit 1d18268 into mui:master Jun 16, 2026
24 checks passed
@alexfauquette

Copy link
Copy Markdown
Member Author

Those tests should probably not change. The computation of the domain is a fairly stable feature. Those are really for the edge cases

mbrookes pushed a commit to mbrookes/mui-x that referenced this pull request Jun 27, 2026
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.

[charts] Setting axis min inside the data range collapses the domain to a point

2 participants