Dialog: Changed height measurement to hide content because IE quirks mode ignores height: 0 (Fixed #4350 No Scrollbars in IE in Quirksmode)

1 files changed, 7 insertions, 7 deletions

diff --git a/ui/jquery.ui.dialog.js b/ui/jquery.ui.dialog.js
index 167ffaca7..6d13935de 100644
--- a/ui/jquery.ui.dialog.js
+++ b/ui/jquery.ui.dialog.js
@@ -547,11 +547,9 @@ $.widget("ui.dialog", {
 		var options = this.options;
 
 		// reset content sizing
-		this.element.css({
-			height: 0,
-			minHeight: 0,
-			width: 'auto'
-		});
+		// hide for non content measurement because height: 0 doesn't work in IE quirks mode (see #4350)
+		this.element.css('width', 'auto')
+			.hide();
 
 		// reset wrapper sizing
 		// determine the height of all the non-content elements
@@ -567,8 +565,10 @@ $.widget("ui.dialog", {
 				height: 'auto'
 			}
 			: {
-				height: Math.max(options.height - nonContentHeight, 0)
-			});
+				minHeight: 0,
+				height: Math.max(options.height - nonContentHeight, 0)				
+			})
+			.show();
 
 		(this.uiDialog.is(':data(resizable)') &&
 			this.uiDialog.resizable('option', 'minHeight', this._minHeight()));