-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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] Hotfix missing styles in dark mode #35179
Conversation
|
Noticed this in MUI X docs as well: https://deploy-preview-6773--material-ui-x.netlify.app/x/api/data-grid/data-grid/ |
@siriwatknp To confirm if I understand this correctly: Once we have the CssVarsProvider on all the pages, we can remove the whole |
Not the whole block (otherwise the dark mode does not work), we could replace the |
@siriwatknp Right, my bad. So more like this?
It sounds good :). Ideally, I think that it would be great to be able to replace the |
Preview: https://deploy-preview-35179--material-ui.netlify.app/material-ui/api/alert/
Reported in #34976 (comment).
After:
My mistake. Forgot the case where the tokens have different values between color modes.
will do a hotfix once the CIs are green.