Fix: Put dialog to open center top instead of center middle

If the jstree is too big vertically the middle page will be strictly outisde of bounds of the user view, which means the dialog will open and the page will scroll to the middle and depending on the size of the page this can become confuse. Also, it sets z-index so the dialog will always be in front of the page Signed-off-by: Bruno Bottazzini <bruno.bottazzini@intel.com>
solettaproject · Sep 16, 2015 · 2168803 · 2168803
1 parent fa24e97
commit 2168803
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/client/css/app.css b/client/css/app.css
@@ -339,3 +339,7 @@ iframe{
 .enable_div {
     display: inline;
 }
+
+.ui-dialog {
+    z-index: 9999 !important ;
+}
diff --git a/client/js/controllers/editor.js b/client/js/controllers/editor.js
@@ -218,7 +218,7 @@
                                 title: "Run",
                                 autoOpen: false,
                                 modal: true,
-                                position: { at: "middle center"},
+                                position: { at: "center top"},
                                 height: 'auto',
                                 width: 650,
                                 show: { effect: "fade", duration: 300 },
@@ -326,7 +326,7 @@
                                     autoOpen: false,
                                     modal: true,
                                     dialogClass: 'no-close',
-                                    position: { at: "middle center"},
+                                    position: { at: "center top"},
                                     height: 60,
                                     width: 50,
                                     show: { effect: "fade", duration: 300 },
@@ -378,7 +378,7 @@
                             title: "Git Commit",
                             autoOpen: false,
                             modal: true,
-                            position: { at: "middle center"},
+                            position: { at: "center top"},
                             height: 443,
                             width: 650,
                             show: { effect: "fade", duration: 300 },
@@ -533,7 +533,7 @@
                             title: "Choose a name",
                             autoOpen: false,
                             modal: true,
-                            position: { at: "middle center"},
+                            position: { at: "center top"},
                             height: 167,
                             width: 270,
                             show: { effect: "fade", duration: 300 },
@@ -563,7 +563,7 @@
                             title: title,
                             autoOpen: false,
                             modal: true,
-                            position: { at: "middle center"},
+                            position: { at: "center top"},
                             height: 220,
                             width: 412,
                             resizable: 'disable',