Skip to content

Add new extra large size (.modal-xl) for .modal-dialog#19925

Closed
malkomich wants to merge 2 commits intotwbs:v4-devfrom
malkomich:v4-dev
Closed

Add new extra large size (.modal-xl) for .modal-dialog#19925
malkomich wants to merge 2 commits intotwbs:v4-devfrom
malkomich:v4-dev

Conversation

@malkomich
Copy link
Copy Markdown
Contributor

No description provided.

@cvrebert cvrebert changed the title Add new extra large size for modal containers Add new extra large size (.modal-xl) for .modal-dialog May 17, 2016
Comment thread docs/components/modal.md Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Missing another button for this around line 187

@cvrebert
Copy link
Copy Markdown
Collaborator

You also need to update line 184 to reflect the addition of your new size. Currently it says "Modals have two optional sizes".

@mdo
Copy link
Copy Markdown
Member

mdo commented May 17, 2016

Why add another one?

@malkomich
Copy link
Copy Markdown
Contributor Author

@mdo Because it was useful for me to display large tables and forms in a modal container, so I guess someone could enjoy this small enhancement.

And also it doesn't break anything..so why not? 😄

@mdo mdo modified the milestone: v4.1 ideas Oct 10, 2016
@ASHFAQPATWARI
Copy link
Copy Markdown

This is nice to have option.

@bogdaniel
Copy link
Copy Markdown

actully i'm working with some modals now and was thinking i should increase the size.

@chroda
Copy link
Copy Markdown

chroda commented Dec 18, 2017

is this issue dead?

Comment thread scss/_modal.scss
@@ -140,3 +140,7 @@
@include media-breakpoint-up(lg) {
.modal-lg { max-width: $modal-lg; }
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Maybe we can add a line below to here for smooth transition between md and lg break points.
.modal-xl { max-width: $modal-lg; }

@chroda
Copy link
Copy Markdown

chroda commented Mar 8, 2018

This is so small and non-disruptive enhancement, guess is taken to long to get aproved the PR

@ghost ghost mentioned this pull request May 11, 2018
@mdo mdo mentioned this pull request Jul 8, 2018
@mdo mdo removed this from the v4.1 ideas milestone Jul 8, 2018
@mdo
Copy link
Copy Markdown
Member

mdo commented Jul 8, 2018

Update based on this PR coming in #26821.

@mdo mdo closed this Jul 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants