Dialog: modified so that minWidth is respected. Fixes #5531 - dialog …

…width should be at least minWidth on creation.
jquery · Jul 30, 2010 · c5770c0 · c5770c0
1 parent 90caa93
commit c5770c0
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 0 deletions.
diff --git a/tests/unit/dialog/dialog_tickets.js b/tests/unit/dialog/dialog_tickets.js
@@ -40,4 +40,25 @@ test("#5184: isOpen in dialogclose event is true", function() {
 	el.remove();
 });
 
+test("#5531: dialog width should be at least minWidth on creation", function () {
+    el = $('<div></div>').dialog({
+            width: 200,
+            minWidth: 300
+        });
+
+    equals(el.dialog('option', 'width'), 300, "width is minWidth");
+    el.dialog('option', 'width', 200);
+    equals(el.dialog('option', 'width'), 300, "width unchanged when set to < minWidth");
+    el.dialog('option', 'width', 320);
+    equals(el.dialog('option', 'width'), 320, "width changed if set to > minWidth");
+    el.remove();
+
+    el = $('<div></div>').dialog({
+            minWidth: 300
+        });
+    ok(el.dialog('option', 'width') >=  300, "width is at least 300");
+    el.remove();
+
+});
+
 })(jQuery);
diff --git a/ui/jquery.ui.dialog.js b/ui/jquery.ui.dialog.js
@@ -628,6 +628,10 @@ $.widget("ui.dialog", {
 			height: 0
 		});
 
+		if (options.minWidth > options.width) {
+			options.width = options.minWidth;
+		}
+
 		// reset wrapper sizing
 		// determine the height of all the non-content elements
 		nonContentHeight = this.uiDialog.css({