mui-x icon indicating copy to clipboard operation
mui-x copied to clipboard

[charts] Fix themeAugmentation

Open alexfauquette opened this issue 1 year ago • 2 comments

Follow up of #14313

I was looking to add describeConformance but those test make assumptions that are not respected by charts. For example it expects ChartsXAxis to propagate its props to the root element. Whereas the component take the needed value from its props and render the axis.

In this PR I took care of the general components. Each component has its dedicated commit

alexfauquette avatar Aug 28 '24 11:08 alexfauquette

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

Generated by :no_entry_sign: dangerJS against 690f1be6ac8963db1e1da497a1f49d95761486bc

mui-bot avatar Aug 28 '24 11:08 mui-bot

CodSpeed Performance Report

Merging #14372 will not alter performance

Comparing alexfauquette:continue-yhemeOverride (690f1be) with master (70b6cb7)

Summary

✅ 3 untouched benchmarks

codspeed-hq[bot] avatar Aug 28 '24 11:08 codspeed-hq[bot]

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

github-actions[bot] avatar Aug 29 '24 09:08 github-actions[bot]