[charts] Allow skipping tooltip render
I did a test with the following component. Seems good
function TestTTooltip() {
const tooltipData = useItemTooltip();
return tooltipData === null ? (
<text x={200} y={200} fill="red">
NULL
</text>
) : (
<ChartsTooltipContainer trigger="item">
<CustomItemTooltip />
</ChartsTooltipContainer>
);
}
Fix #18035
Deploy preview: https://deploy-preview-18050--material-ui-x.netlify.app/
Updated pages:
Bundle size report
Total Size Change:${\tiny{\color{red}▲}}$+2.87KB(+0.02%) - Total Gzip Change:${\tiny{\color{red}▲}}$+1.17KB(+0.03%) Files: 120 total (0 added, 0 removed, 16 changed)
@mui/x-charts-pro/Heatmap parsed:${\tiny{\color{red}▲}}$+383B(+0.21%) gzip:${\tiny{\color{red}▲}}$+99B(+0.16%)
Show 15 more bundle changes
@mui/x-charts parsed:${\tiny{\color{red}▲}}$+180B(+0.07%) gzip:${\tiny{\color{red}▲}}$+97B(+0.12%) @mui/x-charts-pro parsed:${\tiny{\color{red}▲}}$+180B(+0.05%) gzip:${\tiny{\color{red}▲}}$+99B(+0.10%) @mui/x-charts/ChartsTooltip parsed:${\tiny{\color{red}▲}}$+179B(+0.24%) gzip:${\tiny{\color{red}▲}}$+67B(+0.26%) @mui/x-charts-pro/BarChartPro parsed:${\tiny{\color{red}▲}}$+162B(+0.08%) gzip:${\tiny{\color{red}▲}}$+60B(+0.09%) @mui/x-charts-pro/FunnelChart parsed:${\tiny{\color{red}▲}}$+162B(+0.08%) gzip:${\tiny{\color{red}▲}}$+93B(+0.14%) @mui/x-charts-pro/LineChartPro parsed:${\tiny{\color{red}▲}}$+162B(+0.07%) gzip:${\tiny{\color{red}▲}}$+68B(+0.09%) @mui/x-charts-pro/PieChartPro parsed:${\tiny{\color{red}▲}}$+162B(+0.09%) gzip:${\tiny{\color{red}▲}}$+61B(+0.10%) @mui/x-charts-pro/RadarChartPro parsed:${\tiny{\color{red}▲}}$+162B(+0.10%) gzip:${\tiny{\color{red}▲}}$+63B(+0.12%) @mui/x-charts-pro/ScatterChartPro parsed:${\tiny{\color{red}▲}}$+162B(+0.08%) gzip:${\tiny{\color{red}▲}}$+60B(+0.09%) @mui/x-charts/BarChart parsed:${\tiny{\color{red}▲}}$+162B(+0.09%) gzip:${\tiny{\color{red}▲}}$+67B(+0.12%) @mui/x-charts/LineChart parsed:${\tiny{\color{red}▲}}$+162B(+0.09%) gzip:${\tiny{\color{red}▲}}$+74B(+0.12%) @mui/x-charts/RadarChart parsed:${\tiny{\color{red}▲}}$+162B(+0.10%) gzip:${\tiny{\color{red}▲}}$+66B(+0.13%) @mui/x-charts/ScatterChart parsed:${\tiny{\color{red}▲}}$+162B(+0.10%) gzip:${\tiny{\color{red}▲}}$+54B(+0.10%) @mui/x-charts/SparkLineChart parsed:${\tiny{\color{red}▲}}$+162B(+0.09%) gzip:${\tiny{\color{red}▲}}$+68B(+0.12%) @mui/x-charts/PieChart parsed:${\tiny{\color{red}▲}}$+161B(+0.10%) gzip:${\tiny{\color{red}▲}}$+75B(+0.14%)
Generated by :no_entry_sign: dangerJS against ee387874cd26b791375c8c5e956dc69953bb8eb8
CodSpeed Performance Report
Merging #18050 will not alter performance
Comparing alexfauquette:fix-tooltip-early-return (ee38787) with master (3e8ecda)
Summary
✅ 9 untouched benchmarks
@alexfauquette this should be ready for review