Author: jablko
Date: Wed Oct 13 23:33:02 2010
New Revision: 8112

Log:
Since <tr/> is hidden, not removed, be careful about replacing <td/> children 
with &nbsp; when .text() is empty. Instead, always append &nbsp;

Modified:
   trunk/js/multiDelete.js

Modified: trunk/js/multiDelete.js
==============================================================================
--- trunk/js/multiDelete.js     Wed Oct 13 23:26:06 2010        (r8111)
+++ trunk/js/multiDelete.js     Wed Oct 13 23:33:02 2010        (r8112)
@@ -20,16 +20,11 @@
         // play nicely
         $('td:not(:has(.animateNicely))', $parentRows).each(function ()
           {
-            if ('' == jQuery.trim($(this).text()))
-            {
-              // Add a &nbsp; if <td/> has no contents because .hide() doesn't
-              // seem to operate on <div/>s that contain only whitespace
-              $(this).html('<div class="animateNicely">&nbsp;</div>');
-            }
-            else
-            {
-              $(this).wrapInner('<div class="animateNicely"/>');
-            }
+            // Add a &nbsp; if <td/> has no contents because .hide() doesn't
+            // seem to operate on <div/>s that contain only whitespace
+            $(this)
+              .append('&nbsp;')
+              .wrapInner('<div class="animateNicely"/>');
           });
 
         $('div:visible', $parentRows).hide('normal', function ()

-- 
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