Skip to content

[charts] Allow to show only the last mark#21645

Draft
alexfauquette wants to merge 9 commits intomui:masterfrom
alexfauquette:mark-end
Draft

[charts] Allow to show only the last mark#21645
alexfauquette wants to merge 9 commits intomui:masterfrom
alexfauquette:mark-end

Conversation

@alexfauquette
Copy link
Copy Markdown
Member

@alexfauquette alexfauquette commented Mar 6, 2026

While looking at Google Analytics, I found smart the fact to show the last mark, because it allow at the same time to get

  • Having marks to display the type of series without relying only the color
  • avoiding the render tones of marks that disturbe the chart
Capture d’écran du 2026-03-06 12-57-44

While writing this comment I realized it's even smarted, the marks are only here when there is no pointer. when the pointer is active it's the highlight marks that are visible and not the last one

image

@alexfauquette alexfauquette added type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. scope: charts Changes related to the charts. labels Mar 6, 2026
@mui-bot
Copy link
Copy Markdown

mui-bot commented Mar 6, 2026

Deploy preview: https://deploy-preview-21645--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 🔺+173B(+0.05%) 🔺+85B(+0.08%)
@mui/x-charts-pro 🔺+173B(+0.04%) 🔺+78B(+0.05%)
@mui/x-charts-premium 🔺+173B(+0.03%) 🔺+73B(+0.05%)
@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 a691eb3

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Mar 6, 2026

Merging this PR will not alter performance

✅ 14 untouched benchmarks


Comparing alexfauquette:mark-end (a691eb3) with master (9749b4c)

Open in CodSpeed

@github-actions github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Mar 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: out-of-date The pull request has merge conflicts and can't be merged. 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.

2 participants