Skip to content

[Dialog] Add xl maxWidth and demo component#13694

Merged
oliviertassinari merged 2 commits intomui:masterfrom
dispix:dialog-xl-maxWidth
Nov 26, 2018
Merged

[Dialog] Add xl maxWidth and demo component#13694
oliviertassinari merged 2 commits intomui:masterfrom
dispix:dialog-xl-maxWidth

Conversation

@dispix
Copy link
Copy Markdown
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 type: new feature Expand the scope of the product to solve a new problem. scope: dialog Changes related to the dialog. labels Nov 26, 2018
@oliviertassinari oliviertassinari merged commit 072959c into mui:master Nov 26, 2018
@oliviertassinari
Copy link
Copy Markdown
Member

oliviertassinari commented Nov 26, 2018

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

},
});

class FullScreenDialog extends React.Component {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

MaxWidthDialog

fullScreen?: boolean;
fullWidth?: boolean;
maxWidth?: 'xs' | 'sm' | 'md' | 'lg' | false;
maxWidth?: 'xs' | 'sm' | 'md' | 'lg' | 'xl' | false;
Copy link
Copy Markdown
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

scope: dialog Changes related to the dialog. type: new feature Expand the scope of the product to solve a new problem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants