Changeset 4696

Timestamp:

02/08/08 22:03:04 (1 year ago)

Author:

scott.gonzalez

Message:

Fix #2297. Split out overlay CSS options from modal option.

Location:

trunk/ui

Files:

• tests/uiTest.dialog.js (modified) (1 diff)
• ui.dialog.js (modified) (2 diffs)

trunk/ui/tests/uiTest.dialog.js

@@ -42 +42 @@
                 "modal" : {
                     "false" : "$('#foo').dialog({\n\t modal: false \n});",
-                    "true" : "$('#foo').dialog({\n\t modal: true \n});",
-                    "Css-BgWhiteOpacity0.5" : "$('#foo').dialog({\n\t modal: { backgroundColor: 'white', opacity: 0.5 } \n})"
+                    "true" : "$('#foo').dialog({\n\t modal: true \n});"
+                },
+                "overlay" : {
+                    "BgWhiteOpacity0.5" : "$('#foo').dialog({\n\t modal: true, \n\t overlay: { backgroundColor: 'white', opacity: 0.5 } \n})"
                 },
                 "position" : {

trunk/ui/ui.dialog.js

@@ -118 +118 @@
     
         this.open = function() {
-            options.modal && overlay.show(self, options.modal);
+            options.modal && overlay.show(self, options.overlay);
             uiDialog.appendTo('body');
             var wnd = $(window), doc = $(document), top = doc.scrollTop(), left = doc.scrollLeft();
@@ -201 +201 @@
             minWidth: 150,
             modal: false,
+            overlay: {},
             position: 'center',
             resizable: true,