Skip to content
New issue

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

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

Already on GitHub? # to your account

[charts] Add "use client" directive to every public component and hook #14578

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

flaviendelangle
Copy link
Member

@flaviendelangle flaviendelangle commented Sep 11, 2024

Follow up on #14562. Part of #9833.

@alexfauquette @JCQuintas I let you check if some components or hooks can work and be valuable in RSC 👍

@flaviendelangle flaviendelangle added the component: charts This is the name of the generic UI component, not the React module! label Sep 11, 2024
@flaviendelangle flaviendelangle self-assigned this Sep 11, 2024
@flaviendelangle flaviendelangle changed the title [charts] Add use client directive to every public component and hook [charts] Add "use client" directive to every public component and hook Sep 11, 2024
@mui-bot
Copy link

mui-bot commented Sep 11, 2024

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

Generated by 🚫 dangerJS against 0f51bdc

Copy link

codspeed-hq bot commented Sep 11, 2024

CodSpeed Performance Report

Merging #14578 will not alter performance

Comparing flaviendelangle:use-client-charts (0f51bdc) with master (1939c1c)

Summary

✅ 3 untouched benchmarks

@flaviendelangle flaviendelangle merged commit 4f2a244 into mui:master Sep 11, 2024
21 checks passed
@flaviendelangle flaviendelangle deleted the use-client-charts branch September 11, 2024 12:40
@@ -1,3 +1,4 @@
'use client';
Copy link
Member

@oliviertassinari oliviertassinari Sep 12, 2024

Choose a reason for hiding this comment

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

I'm not sure here: mui/material-ui#41956 (comment).

Edit: no, we are good mui/material-ui#43778.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
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.

4 participants