Dialog was incorrectly sized and closed even on an error on save click.

refinery · Jul 23, 2010 · 924b962 · 924b962
1 parent 37fc1af
commit 924b962
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/public/javascripts/refinery/admin.js b/public/javascripts/refinery/admin.js
@@ -547,7 +547,8 @@ var page_options = {
       modal: true,
       resizable: false,
       autoOpen: false,
-      width: 600
+      width: 600,
+      height: 200
     });
 
     $('#add_page_part').click(function(e){
@@ -591,6 +592,8 @@ var page_options = {
               $('#new_page_part_title').val('');
 
               page_options.tabs.find('> ul li a').corner('top 5px');
+
+              $('#new_page_part_dialog').dialog('close');
             }
           );
         }else{
@@ -599,9 +602,6 @@ var page_options = {
       }else{
         alert("You have not entered a title for the content section, please enter one.");
       }
-
-
-      $('#new_page_part_dialog').dialog('close');
     });
 
     $('#new_page_part_cancel').click(function(e){

diff --git a/public/stylesheets/refinery/refinery.css b/public/stylesheets/refinery/refinery.css
@@ -1221,7 +1221,7 @@ ul#plugins li {
   width:auto;
 }
 #new_page_part_dialog .field {
-  padding: 30px 10px;
+  padding: 0px 10px;
 }
 .hide-overflow {
   overflow: hidden;