Skip to content

Conversation

@probins
Copy link

@probins probins commented Aug 18, 2012

Dialog is currently setting maxHeight on resizable but not on initial open. I think this is all that's necessary to fix this.

Should resolve http://bugs.jqueryui.com/ticket/4820

Dialog is currently setting maxHeight on resizable but not on initial open. I think this is all that's necessary to fix this. Should resolve http://bugs.jqueryui.com/ticket/4820
@scottgonzalez
Copy link
Member

This is incomplete and incorrect. maxHeight is for the entire dialog, including the wrapper. You're setting the height directly on the content element, which will result in the wrong size. You're also not handling IE6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants