Dialog: Moved positioning and overflow to CSS. Fixes #2848 - Fixed positioning for...

diff --git a/themes/base/ui.dialog.css b/themes/base/ui.dialog.css
index 2997595b7d276e06108270ee0ac45c608d260439..609f6f984e48b218e59985a3622bba37993fa379 100644 (file)
--- a/themes/base/ui.dialog.css
+++ b/themes/base/ui.dialog.css
@@ -1,12 +1,12 @@
 /* Dialog
 ----------------------------------*/
-.ui-dialog { position: relative; padding: .2em; width: 300px; }
+.ui-dialog { position: absolute; padding: .2em; width: 300px; overflow: hidden; }
 .ui-dialog .ui-dialog-titlebar { padding: .5em .3em .3em 1em; position: relative;  }
 .ui-dialog .ui-dialog-title { float: left; margin: .1em 0 .2em; } 
 .ui-dialog .ui-dialog-titlebar-close { position: absolute; right: .3em; top: 50%; width: 19px; margin: -10px 0 0 0; padding: 1px; height: 18px; }
 .ui-dialog .ui-dialog-titlebar-close span { display: block; margin: 1px; }
 .ui-dialog .ui-dialog-titlebar-close:hover, .ui-dialog .ui-dialog-titlebar-close:focus { padding: 0; }
-.ui-dialog .ui-dialog-content { border: 0; padding: .5em 1em; background: none; overflow: auto; zoom: 1; }
+.ui-dialog .ui-dialog-content { position: relative; border: 0; padding: .5em 1em; background: none; overflow: auto; zoom: 1; }
 .ui-dialog .ui-dialog-buttonpane { text-align: left; border-width: 1px 0 0 0; background-image: none; margin: .5em 0 0 0; padding: .3em 1em .5em .4em; }
 .ui-dialog .ui-dialog-buttonpane button { float: right; margin: .5em .4em .5em 0; cursor: pointer; padding: .2em .6em .3em .6em; line-height: 1.4em; width:auto; overflow:visible; }
 .ui-dialog .ui-resizable-se { width: 14px; height: 14px; right: 3px; bottom: 3px; }

diff --git a/ui/ui.dialog.js b/ui/ui.dialog.js
index e0955664b95be940fdd1bca7c897304684a13711..7a373f97707b73d9eb0d553241af3ee052500db2 100644 (file)
--- a/ui/ui.dialog.js
+++ b/ui/ui.dialog.js
@@ -43,8 +43,6 @@ $.widget("ui.dialog", {
                                .hide()
                                .addClass(uiDialogClasses + options.dialogClass)
                                .css({
-                                       position: 'absolute',
-                                       overflow: 'hidden',
                                        zIndex: options.zIndex
                                })
                                // setting tabIndex makes the div focusable