From c2598fe4d85f9c3b737ea40959f44bf04172eafa Mon Sep 17 00:00:00 2001
From: Chad Kieffer <chad@2tbsp.com>
Date: Thu, 18 Dec 2008 02:50:40 +0000
Subject: Dialog polishing. They now resize their height. Updated form styles
 in the context of dialog display (don't show legends, fieldset border, or
 submit button).

Let's make sure that all forms generated are complete and well formed and then use JS and CSS to control their display and behavior in different contexts.
---
 themes/default/js/ui.init.js | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

(limited to 'themes/default/js/ui.init.js')

diff --git a/themes/default/js/ui.init.js b/themes/default/js/ui.init.js
index 3f917ff0..7a9c69b5 100644
--- a/themes/default/js/ui.init.js
+++ b/themes/default/js/ui.init.js
@@ -95,6 +95,8 @@ function handleDialogEvent(event) {
  * @see handleDialogEvent()
  * 
  * @todo Set dialog attributes dynamically (width, height, drag, resize)
+ * @todo Set ui-dialog-buttonpane button values equal to the original form button value
+ * @todo Display loading animation on form submit
  */
 function openDialog(element) {
   var sHref = $(element).attr("href");
@@ -124,7 +126,7 @@ function openDialog(element) {
   $("#gDialog").dialog({
     autoResize: false,
     draggable: true,
-    height: 500,
+    height: $(window).height() - 40,
     modal: true,
     overlay: {
       opacity: 0.7,
@@ -144,6 +146,18 @@ function openDialog(element) {
   $.get(sHref, function(data) {
     $("#gDialog").removeClass("gLoadingLarge");	
     $("#gDialog").html(data).hide().fadeIn();
+    // Get dialog and it's contents' height
+    var contentHt =  $(".ui-dialog-titlebar").height() 
+        + $(".ui-dialog-content form").height() 
+        + $(".ui-dialog-buttonpane").height() 
+        + 60;
+    // Resize height if content's shorter than dialog        
+    if (contentHt < $("#gDialog").data("height.dialog")) {
+      $(".ui-dialog").animate({height: contentHt});
+    }
   });
+  
+  
+  
   return false;
 }
-- 
cgit v1.2.3