-
Notifications
You must be signed in to change notification settings - Fork 832
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
Feature theme augmentation #2092
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/mui-org/material-ui-pickers/mb1m25znv |
@rajzik Thanks for the effort. Note that we are working on merging the components into the lab in https://github.com/mui-org/core-plus-pickers. We will have to redo this pull request on https://github.com/mui-org/material-ui once the migration is completed. |
@oliviertassinari oh, nvm. If you need any help just let me know. |
I will let you know :) |
@oliviertassinari is there any estimation when the migration will be done? |
@dmtrKovalenko is making great progress. I think that so far, the main blocker is to reach a point where https://github.com/mui-org/core-plus-pickers/tree/next/packages/pickers is no longer included in the repository. |
@rajzik Thanks for the effort! I believe we have solved the issue by moving the component in the lab. |
Fixes: #1874
This PR will unify the approach to the material-ui monorepo see: https://github.com/mui-org/material-ui/tree/next/packages/material-ui-lab/src/themeAugmentation
I also added documentation for the theme augmentation as well as fixed component names to correspond with theme overrides.