Skip to content

[lab] Add use client directive#40358

Merged
DiegoAndai merged 2 commits intomui:masterfrom
DiegoAndai:mui-lab-use-client
Jan 2, 2024
Merged

[lab] Add use client directive#40358
DiegoAndai merged 2 commits intomui:masterfrom
DiegoAndai:mui-lab-use-client

Conversation

@DiegoAndai
Copy link
Member

@DiegoAndai DiegoAndai commented Dec 29, 2023

Add @mui/lab to the RSC build step and run it for the first time, adding the corresponding 'use client' directives.

Fix #40349

@DiegoAndai DiegoAndai added package: lab Specific to the lab. internal Behind-the-scenes enhancement. Formerly called “core”. labels Dec 29, 2023
@DiegoAndai DiegoAndai self-assigned this Dec 29, 2023
@mui-bot
Copy link

mui-bot commented Dec 29, 2023

Netlify deploy preview

https://deploy-preview-40358--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 6df98ce

@oliviertassinari
Copy link
Member

oliviertassinari commented Dec 29, 2023

Nice 👍


Off-topic

It would be great to remove all the directives from the index files as well, e.g.

It makes it hard to expose server side logic from the npm package, e.g. #42750

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

Labels

internal Behind-the-scenes enhancement. Formerly called “core”. nextjs package: lab Specific to the lab.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[material-ui][Timeline] Not working with server-side in an App Router Next.js project

4 participants