inner media should have a max-width and max-height value #2

anselmh · 2013-01-25T16:51:18Z

drublic · 2013-01-25T16:53:40Z

max-width seems a good solution. max-height does not make sense I think because you can scroll the content-area.
What do you think?

anselmh · 2013-01-25T16:54:08Z

True. Only max-width then.

anselmh · 2013-01-25T16:57:54Z

You also probably be a bit more flexible using this additional line of code:

.modal-content {
    max-height: 400px;
    max-height: 70vh;
}

Although vh should be 90vh to the whole modal I think. This 70vh is a quick fix for current base.

Reference #2.

ghost assigned drublic Jan 25, 2013

drublic closed this as completed in 79f4162 Jan 26, 2013

drublic added a commit that referenced this issue Jan 26, 2013

Set max-height for modal-content in vh

5f7bfac

Reference #2.

Provide feedback