Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[charts-pro] Fix pro components watermark #16222

Merged
merged 18 commits into from
Jan 20, 2025

Conversation

JCQuintas
Copy link
Member

@JCQuintas JCQuintas commented Jan 17, 2025

  • Added ChartDataProviderPro which checks and displays the watermark.
  • Removed watermark from ChartsContainerPro as it uses ChartDataProviderPro under the hood
  • Moved /context/ChartDataProvider to /ChartDataProvider to be easier to use
  • Added tests to prevent this regression

@JCQuintas JCQuintas added bug 🐛 Something doesn't work component: charts This is the name of the generic UI component, not the React module! labels Jan 17, 2025
@JCQuintas JCQuintas self-assigned this Jan 17, 2025
@mui-bot
Copy link

mui-bot commented Jan 17, 2025

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

Updated pages:

Generated by 🚫 dangerJS against 912a743

Copy link

codspeed-hq bot commented Jan 17, 2025

CodSpeed Performance Report

Merging #16222 will not alter performance

Comparing JCQuintas:fix-data-provider-pro (912a743) with master (2eacdb0)

Summary

✅ 6 untouched benchmarks

Comment on lines 58 to 60
{ "name": "ChartDataProviderPro", "kind": "Function" },
{ "name": "ChartDataProviderProProps", "kind": "TypeAlias" },
{ "name": "ChartDrawingArea", "kind": "TypeAlias" },
Copy link
Member

Choose a reason for hiding this comment

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

I would be in favor of adding an export of the MIT container in packages/x-charts-pro/src/index.ts such that pro users can both have the provider with and without zoom

@JCQuintas JCQuintas merged commit 198a1f6 into mui:master Jan 20, 2025
17 checks passed
@JCQuintas JCQuintas deleted the fix-data-provider-pro branch January 20, 2025 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: charts This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants