-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[docs-infra] Revert to use deprecated CssVarsProvider
for MUI X
#45371
[docs-infra] Revert to use deprecated CssVarsProvider
for MUI X
#45371
Conversation
siriwatknp
commented
Feb 20, 2025
- I have followed (at least) the PR section of the contributing guide.
Netlify deploy previewhttps://deploy-preview-45371--material-ui.netlify.app/ Bundle size report |
…-infra/temp-cssvarsprovider
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix
Experimental_CssVarsProvider as CssVarsProvider, | ||
// @ts-expect-error need to use deprecated API because MUI X repo still on Material UI v5 | ||
experimental_extendTheme as extendTheme, | ||
createColorScheme, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not seem to work. 🙈
https://app.circleci.com/pipelines/github/mui/mui-x/82297/workflows/ffc58350-4471-4c39-a160-357f1cdd36da/jobs/468718?invite=true#step-109-2430_145
Looks like it's already time to bump @mui/material
in docs. 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will find a workaround. It should be doable to avoid the type error.