Allow theme switcher override with slots#4340
Allow theme switcher override with slots#4340apedroferreira merged 7 commits intomui:masterfrom apedroferreira:theme-switcher-slots
Conversation
… components & add custom theme switcher example
github-actions
bot
added
the
PR: out-of-date
bharatkashyap
left a comment
bharatkashyap
left a comment
There was a problem hiding this comment.
Looks good! The demo and explanation of the slots seems clear to me. Perhaps we can have a diagram eventually when we have more slots
Yeah, will kind of see how it evolves as we go, the new "Examples" section should help show some particular scenarios that might be useful. |
github-actions
bot
removed
the
PR: out-of-date
|
@bharatkashyap I just noticed this if you sign out in the "Account in sidebar" example: 
Probably no need to fix it in this release, just ran into it! Also not sure how to best handle it in the mini variant of the sidebar... |
Thanks for that catch! I'll open an issue to track this |
3 participants
Closes #4151 and #4291
ThemeSwitchercomponent intoolbarActionsslot so it can be hiddenThemeSwitcherandToolbarActions(default slot) componentsDashboardLayoutdemo page in docshttps://deploy-preview-4340--mui-toolpad-docs.netlify.app/toolpad/core/react-dashboard-layout/#customization