-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[Dialog] Add a dividers boolean prop #14795
Conversation
Details of bundle changes.Comparing: 099bc5e...2f4cdd9
|
0c9f4e3
to
2f4cdd9
Compare
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.
@oliviertassinari Forgot to submit review. Only concerned with missing changelog entries.
return ( | ||
<Typography component="p" variant="subtitle1" color="textSecondary" ref={ref} {...props} /> | ||
); | ||
return <Typography component="p" variant="body1" color="textSecondary" ref={ref} {...props} />; |
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.
breaking change, needs changelog attention.
}, | ||
/* Styles applied to the children. */ | ||
/* Styles applied to the root element if `disableActionSpacing={false}`. */ |
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.
breaking change, needs changelog attention.
* [docs] Add missing breaking changes from #14795 * fals€ -> false * remove pull request * Update docs/src/pages/guides/migration-v3/migration-v3.md Co-Authored-By: eps1lon <[email protected]> * Update docs/src/pages/guides/migration-v3/migration-v3.md Co-Authored-By: eps1lon <[email protected]>
Closes #3155
Closes #14735
Closes #14777
Breaking change
Change the Dialog spacings to better match the specification: https://material.io/design/components/dialogs.html.