Skip to content

[docs] Fix @global being considered a class#22297

Merged
oliviertassinari merged 2 commits intomui:nextfrom
eps1lon:docs/fix-global-classkey
Aug 21, 2020
Merged

[docs] Fix @global being considered a class#22297
oliviertassinari merged 2 commits intomui:nextfrom
eps1lon:docs/fix-global-classkey

Conversation

@eps1lon
Copy link
Member

@eps1lon eps1lon commented Aug 21, 2020

BREAKING CHANGE

Remove deprecated CssBaselineClassKey TypeScript type. Using this class key always generated a runtime dev-only warning.

Noticed that we wrongly generate a @global class in the docs for InputBase. We already deprecated that class for CSSBaseline so we might as well fix the overall issue.

The mention of default props in theme for CSSBaseline is not that useful since there's only children. I still kept it for consistency. In the end withStyles currently does two things: default props override via theme and styling.

@eps1lon eps1lon added docs Improvements or additions to the documentation. breaking change Introduces changes that are not backward compatible. typescript labels Aug 21, 2020
@mui-pr-bot
Copy link

mui-pr-bot commented Aug 21, 2020

Details of bundle changes

Generated by 🚫 dangerJS against f8e7f89

@oliviertassinari oliviertassinari merged commit 7329e94 into mui:next Aug 21, 2020
@eps1lon eps1lon deleted the docs/fix-global-classkey branch August 21, 2020 18:40
eps1lon added a commit to eps1lon/material-ui that referenced this pull request Aug 22, 2020
@eps1lon eps1lon added this to the v5 milestone Sep 15, 2020
@oliviertassinari oliviertassinari mentioned this pull request Sep 15, 2020
42 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Introduces changes that are not backward compatible. docs Improvements or additions to the documentation. typescript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants