Changeset 36394 for trunk/src/wp-admin/includes/class-wp-ms-themes-list-table.php

Timestamp:

01/25/2016 10:54:52 PM (4 years ago)

Author:

obenland

Message:

Pass data consistently on plugin, network plugin, and network theme screens.

List tables on these three screens look similar but come with unexpected caveats.
The network themes screen in particular has been neglected, using a .plugin
class to inherit its sibling's styles and not providing classes and data
attributes that the other two offer. This will provide some more consistency.

Fixes #35335.

File:

• trunk/src/wp-admin/includes/class-wp-ms-themes-list-table.php (modified) (1 diff)

trunk/src/wp-admin/includes/class-wp-ms-themes-list-table.php

@@ -683 +683 @@
 
         $class = ! $allowed ? 'inactive' : 'active';
-
-        $id = sanitize_html_class( $theme->get_stylesheet() );
-
         if ( ! empty( $totals['upgrade'] ) && ! empty( $theme->update ) ) {
             $class .= ' update';
         }
 
-        echo "<tr id='$id' class='$class'>";
+        printf( '<tr class="%s" data-slug="%s">',
+            esc_attr( $class ),
+            esc_attr( $stylesheet )
+        );
 
         $this->single_row_columns( $theme );