Changeset 34179

Timestamp:

09/15/2015 01:43:28 PM (9 years ago)

Author:

SergeyBiryukov

Message:

Comments: When setting the colspan value for inline edit/reply, make sure it's actually a table and the value is not empty.

Props afercia.
Fixes #33596.

File:

• trunk/src/wp-admin/js/edit-comments.js (modified) (2 diffs)

trunk/src/wp-admin/js/edit-comments.js

@@ -624 +624 @@
             t = this,
             c = $('#comment-' + comment_id),
-            h = c.height();
+            h = c.height(),
+            colspanVal = 0;
 
         t.close();
@@ -634 +635 @@
         act = 'edit' == action ? 'edit' : 'replyto';
         act = t.act = act + '-comment';
-
-        $( 'td', editRow ).attr( 'colspan', $( 'th:visible, td:visible', c ).length );
+        colspanVal = $( 'th:visible, td:visible', c ).length;
+
+        // Make sure it's actually a table and there's a `colspan` value to apply.
+        if ( editRow.hasClass( 'inline-edit-row' ) && 0 !== colspanVal ) {
+            $( 'td', editRow ).attr( 'colspan', colspanVal );
+        }
 
         $('#action', editRow).val(act);