Skip to content

[AppProvider] Make theming not require specific HTML attribute name#4910

Merged
apedroferreira merged 3 commits intomui:masterfrom
apedroferreira:improve-theming
Apr 30, 2025
Merged

[AppProvider] Make theming not require specific HTML attribute name#4910
apedroferreira merged 3 commits intomui:masterfrom
apedroferreira:improve-theming

Conversation

@apedroferreira
Copy link
Collaborator

@apedroferreira apedroferreira commented Apr 29, 2025

I realized that we probably were requiring the CSS selector in the theme to be data-toolpad-color-scheme but this shouldn't be necessary...

Also removed a line from the theming docs that should be outdated by now.

@apedroferreira apedroferreira added docs Improvements or additions to the documentation. customization: theme Higher level theming customizability. labels Apr 29, 2025
@apedroferreira apedroferreira self-assigned this Apr 29, 2025
@apedroferreira apedroferreira requested a review from a team April 29, 2025 15:48
@apedroferreira apedroferreira moved this to In progress in Toolpad public roadmap Apr 29, 2025
@mui-bot
Copy link

mui-bot commented Apr 29, 2025

Netlify deploy preview

https://deploy-preview-4910--mui-toolpad-docs.netlify.app/

Generated by 🚫 dangerJS against c82ca43

@apedroferreira apedroferreira changed the title [AppProvider] Make it so that theming doesn't require a specific HTML attribute name [AppProvider] Make theming not require specific HTML attribute name Apr 29, 2025
@apedroferreira apedroferreira merged commit f574abd into mui:master Apr 30, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this from In progress to Completed in Toolpad public roadmap Apr 30, 2025
@apedroferreira apedroferreira deleted the improve-theming branch April 30, 2025 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

customization: theme Higher level theming customizability. docs Improvements or additions to the documentation.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants