Changeset 63398

Timestamp:

08/30/2026 09:40:15 AM (6 weeks ago)

Author:

afercia

Message:

Administration: Render a table row header in the Application Passwords AJAX row template.

Tables extending WP_List_Table use a <th scope="row"> element as the row
header for the primary column. WP_List_Table::single_row_columns() renders
this for server-generated rows, but the JavaScript template used when a new
application password is created via AJAX rendered every cell as a <td> element,
leaving the row header missing until the page was reloaded.

Renders the primary column of the JavaScript row template as a
<th scope="row"> element, mirroring the server-generated markup.

Props khokansardar, mukesh27, afercia.
See #32892.
Fixes #65707.

Location:

trunk

Files:

• src/wp-admin/includes/class-wp-application-passwords-list-table.php (modified) (2 diffs)
• tests/phpunit/tests/admin/wpApplicationPasswordsListTable.php (added)

trunk/src/wp-admin/includes/class-wp-application-passwords-list-table.php

@@ -215 +215 @@
                         }
 
-                        printf( '<td class="%s" data-colname="%s">', esc_attr( $classes ), esc_attr( wp_strip_all_tags( $display_name ) ) );
+                        // The primary column is a `<th scope="row">` row header, matching the
+                        // server-rendered markup in WP_List_Table::single_row_columns().
+                        $tag   = $is_primary ? 'th' : 'td';
+                        $scope = $is_primary ? ' scope="row"' : '';
+
+                        printf( '<%1$s class="%2$s" data-colname="%3$s"%4$s>', $tag, esc_attr( $classes ), esc_attr( wp_strip_all_tags( $display_name ) ), $scope );
 
                         switch ( $column_name ) {
@@ -260 +265 @@
                         }
 
-                        echo '</td>';
+                        echo "</{$tag}>";
                 }