-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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
[Icons] deprecate default
value for fontSize
in favor of medium
#23951
Conversation
Details of bundle changes.Comparing: b0e10a1...d813865 Details of page changes
|
@@ -15,7 +15,7 @@ export interface SvgIconTypeMap<P = {}, D extends React.ElementType = 'svg'> { | |||
/** | |||
* The fontSize applied to the icon. Defaults to 24px, but can be configure to inherit font size. | |||
*/ | |||
fontSize?: 'inherit' | 'default' | 'small' | 'large'; | |||
fontSize?: 'inherit' | 'large' | 'medium' | 'small'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is a deprecation, shouldn't we have both default
and medium
to allow migration? Otherwise, we are breaking the build with this change.
Deprecations should be applied on the |
@mnajdova thanks. Where's the "quick, hide this PR so no-one notices the copy-pasta" button? |
@oliviertassinari "Deprecations should be applied on the v4-deprecations branch." I only have myself to blame for that. "It's simple" I said. 😆 |
Co-authored-by: Marija Najdova <mnajdova@gmail.com>
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/mui-org/material-ui/egqzfq3td |
Closing to change target branch |
Deprecation for #23950