Author: jablko
Date: Thu Oct 21 22:01:50 2010
New Revision: 8370

Log:
Double <div/> wrapper not necessary

Modified:
   trunk/js/dialog.js

Modified: trunk/js/dialog.js
==============================================================================
--- trunk/js/dialog.js  Thu Oct 21 21:51:32 2010        (r8369)
+++ trunk/js/dialog.js  Thu Oct 21 22:01:50 2010        (r8370)
@@ -45,12 +45,12 @@
               case 'radio':
               case 'checkbox':
                 thisDialog.fields[this.name] = [];
-                var $input = $('input[name=' + this.name + ']', 
thisDialog.table).each(function ()
-                  {
-                    thisDialog.fields[this.name].push(this);
-                  });
+                thisDialog.initialValues[this.name] = $('input[name=' + 
this.name + ']', thisDialog.table).each(function ()
+                    {
+                      thisDialog.fields[this.name].push(this);
+                    })
+                  .filter(':checked').val();
 
-                thisDialog.initialValues[this.name] = 
$input.filter(':checked').val();
                 break;
 
               default:
@@ -60,17 +60,15 @@
           });
 
         // Create YUI container for dialog
-        var $yuiDialogWrapper = $('<div class="yui-skin-sam">'
-          + '  <div id="' + this.table.id + '">'
-          + '    <div class="hd">'
-          + '      ' + this.label
-          + '    </div><div class="bd">'
-          + '      <form action="" method="post" style="border: none"></form>'
-          + '    </div>'
+        var $yuiDialogWrapper = $('<div id="' + this.table.id + '">'
+          + '  <div class="hd">'
+          + '    ' + this.label
+          + '  </div><div class="bd">'
+          + '    <form action="" method="post" style="border: none"></form>'
           + '  </div>'
           + '</div>');
 
-        // Bind onClick event to "Add" link
+        // Bind click event to "Add" link
         var $addLink = $('<a href="#">Add new</a>').click(function (event)
           {
             // Prevent default action, "go to top of page"
@@ -173,7 +171,7 @@
             }
           }
 
-        // Bind onSubmit method
+        // Bind submit method
         this.$form.submit(this.onSubmit);
 
         // Wait for all iframes to finish before submitting main form

-- 
You received this message because you are subscribed to the Google Groups 
"Qubit Toolkit Commits" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/qubit-commits?hl=en.

Reply via email to