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

[core] True ESM support #43938

Open
1 of 15 tasks
oliviertassinari opened this issue Sep 29, 2024 · 3 comments
Open
1 of 15 tasks

[core] True ESM support #43938

oliviertassinari opened this issue Sep 29, 2024 · 3 comments
Assignees
Labels
breaking change scope: code-infra Specific to the core-infra product umbrella For grouping multiple issues to provide a holistic view

Comments

@oliviertassinari
Copy link
Member

oliviertassinari commented Sep 29, 2024

Steps to reproduce

I'm creating this issue as an umbrella to solve ESM support in Material UI v7.

What does success looks like?

Current behavior

No response

Expected behavior

No response

Context

No response

Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: -

@oliviertassinari oliviertassinari added umbrella For grouping multiple issues to provide a holistic view breaking change status: waiting for maintainer These issues haven't been looked at yet by a maintainer v7.x labels Sep 29, 2024
@oliviertassinari oliviertassinari added scope: code-infra Specific to the core-infra product and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer v7.x labels Sep 29, 2024
@DiegoAndai
Copy link
Member

One of the proposals in #35233 was changing the icons package to be ESM only. This would help reduce the size of the package to 2/3 of its current size (keep type definitions and ESM, remove CJS). Should we do that for v7? @Janpot

@Janpot
Copy link
Member

Janpot commented Dec 19, 2024

for v7 I'd like to go to dual mode packages with exports field and a single layout that's the same across all packages. Personally I think if we go ESM only with one package we should do it for all. The less we confuse bundlers, the better 🙂

@DiegoAndai
Copy link
Member

That makes sense 👍🏼

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change scope: code-infra Specific to the core-infra product umbrella For grouping multiple issues to provide a holistic view
Projects
Status: In progress
Development

No branches or pull requests

3 participants