Skip to content
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

Merged
merged 1 commit into from
Mar 9, 2019

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Mar 8, 2019

Closes #3155
Closes #14735
Closes #14777

Breaking change

Change the Dialog spacings to better match the specification: https://material.io/design/components/dialogs.html.

@oliviertassinari oliviertassinari added breaking change component: dialog This is the name of the generic UI component, not the React module! labels Mar 8, 2019
@mui-pr-bot
Copy link

mui-pr-bot commented Mar 8, 2019

Details of bundle changes.

Comparing: 099bc5e...2f4cdd9

bundle parsed diff gzip diff prev parsed current parsed prev gzip current gzip
@material-ui/core +0.05% 🔺 +0.02% 🔺 362,378 362,566 92,048 92,071
@material-ui/core/Paper 0.00% 0.00% 68,773 68,773 20,155 20,155
@material-ui/core/Paper.esm 0.00% 0.00% 63,294 63,294 19,274 19,274
@material-ui/core/Popper 0.00% 0.00% 30,458 30,458 10,557 10,557
@material-ui/core/styles/createMuiTheme 0.00% 0.00% 17,284 17,284 5,697 5,697
@material-ui/core/useMediaQuery 0.00% 0.00% 2,486 2,486 1,049 1,049
@material-ui/lab 0.00% 0.00% 170,613 170,613 50,038 50,038
@material-ui/styles 0.00% 0.00% 57,118 57,118 16,253 16,253
@material-ui/system 0.00% 0.00% 17,041 17,041 4,498 4,498
Button 0.00% 0.00% 91,106 91,106 26,915 26,915
Modal 0.00% 0.00% 90,214 90,214 26,715 26,715
colorManipulator 0.00% 0.00% 3,232 3,232 1,298 1,298
docs.landing 0.00% 0.00% 51,908 51,908 11,302 11,302
docs.main 0.00% 0.00% 646,179 646,179 200,782 200,782
packages/material-ui/build/umd/material-ui.production.min.js +0.05% 🔺 +0.04% 🔺 310,946 311,116 85,146 85,182

Generated by 🚫 dangerJS against 2f4cdd9

@oliviertassinari oliviertassinari changed the title [Dialog] Add a scrollable boolean prop [WIP][Dialog] Add a scrollable boolean prop Mar 8, 2019
@oliviertassinari oliviertassinari changed the title [WIP][Dialog] Add a scrollable boolean prop [Dialog] Add a dividers boolean prop Mar 8, 2019
@oliviertassinari oliviertassinari added the new feature New feature or request label Mar 8, 2019
@oliviertassinari oliviertassinari requested a review from eps1lon March 8, 2019 10:48
@oliviertassinari oliviertassinari merged commit c36166f into mui:next Mar 9, 2019
@oliviertassinari oliviertassinari deleted the dialog-scrollable branch March 9, 2019 09:46
Copy link
Member

@eps1lon eps1lon left a 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} />;
Copy link
Member

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}`. */
Copy link
Member

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.

eps1lon added a commit to eps1lon/material-ui that referenced this pull request Mar 10, 2019
oliviertassinari pushed a commit that referenced this pull request Mar 11, 2019
* [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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change component: dialog This is the name of the generic UI component, not the React module! new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants