Changeset 33015

Timestamp:

07/01/2015 01:05:29 AM (9 years ago)

Author:

helen

Message:

Bulk/quick edit: Set the correct colspan.

thead can contain tds now, too.

props afercia.
fixes #31654.

Location:

trunk/src/wp-admin/js

Files:

• inline-edit-post.js (modified) (2 diffs)
• inline-edit-tax.js (modified) (1 diff)

trunk/src/wp-admin/js/inline-edit-post.js

@@ -82 +82 @@
         this.revert();
 
-        $('#bulk-edit td').attr('colspan', $('.widefat:first thead th:visible').length);
+        $( '#bulk-edit td' ).attr( 'colspan', $( 'th:visible, td:visible', '.widefat:first thead' ).length );
         // Insert the editor at the top of the table with an empty row above to maintain zebra striping.
         $('table.widefat tbody').prepend( $('#bulk-edit') ).prepend('<tr class="hidden"></tr>');
@@ -132 +132 @@
         // add the new edit row with an extra blank row underneath to maintain zebra striping.
         editRow = $('#inline-edit').clone(true);
-        $('td', editRow).attr('colspan', $('.widefat:first thead th:visible').length);
+        $( 'td', editRow ).attr( 'colspan', $( 'th:visible, td:visible', '.widefat:first thead' ).length );
 
         $(t.what+id).hide().after(editRow).after('<tr class="hidden"></tr>');

trunk/src/wp-admin/js/inline-edit-tax.js

@@ -55 +55 @@
 
         editRow = $('#inline-edit').clone(true), rowData = $('#inline_'+id);
-        $('td', editRow).attr('colspan', $('.widefat:first thead th:visible').length);
+        $( 'td', editRow ).attr( 'colspan', $( 'th:visible, td:visible', '.widefat:first thead' ).length );
 
         $(t.what+id).hide().after(editRow).after('<tr class="hidden"></tr>');