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 xl maxWidth and demo component #13694

Merged
merged 2 commits into from
Nov 26, 2018

Conversation

dispix
Copy link
Contributor

@dispix dispix commented Nov 26, 2018

This PR:

  • Adds support for the XL theme breakpoint to the Dialog component.
  • Adds a demo component in the Dialog demo. The user has the ability to play with the value of both fullWidth and maxWidth to help him understand how they interact.

@oliviertassinari oliviertassinari self-assigned this Nov 26, 2018
@oliviertassinari oliviertassinari added new feature New feature or request component: dialog This is the name of the generic UI component, not the React module! labels Nov 26, 2018
@oliviertassinari oliviertassinari merged commit 072959c into mui:master Nov 26, 2018
@oliviertassinari
Copy link
Member

oliviertassinari commented Nov 26, 2018

@dispix Awesome! I'm fixing the two small issues.

},
});

class FullScreenDialog extends React.Component {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MaxWidthDialog

@@ -9,7 +9,7 @@ export interface DialogProps
children?: React.ReactNode;
fullScreen?: boolean;
fullWidth?: boolean;
maxWidth?: 'xs' | 'sm' | 'md' | 'lg' | false;
maxWidth?: 'xs' | 'sm' | 'md' | 'lg' | 'xl' | false;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing TypeScript key

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants