Changeset 32644 for trunk/src/wp-admin/includes/class-wp-plugins-list-table.php

Timestamp:

05/29/2015 02:40:52 AM (10 years ago)

Author:

helen

Message:

List tables: introduce the concept of a "primary" column.

This becomes the column that contains the row actions, and allows for a more flexibility, particularly with custom post types and list tables. To (re)define the primary column, use the list_table_primary_column filter, which receives the column name and the screen ID as arguments.

props stephdau, DaveAl, jesin.
see #25408.

File:

• trunk/src/wp-admin/includes/class-wp-plugins-list-table.php (modified) (5 diffs)

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

@@ -569 +569 @@
         );
 
-        list( $columns, $hidden ) = $this->get_column_info();
+        list( $columns, $hidden, $sortable, $primary ) = $this->get_column_info();
+
+        $extra_class = ' has-row-actions column-primary';
 
         foreach ( $columns as $column_name => $column_display_name ) {
             $style = '';
-            if ( in_array( $column_name, $hidden ) )
+            if ( in_array( $column_name, $hidden ) ) {
                 $style = ' style="display:none;"';
+            }
 
             switch ( $column_name ) {
@@ -581 +584 @@
                     break;
                 case 'name':
-                    echo "<td class='plugin-title'$style><strong>$plugin_name</strong>";
-                    echo $this->row_actions( $actions, true );
+                    if ( $primary === $column_name || ! isset( $columns[ $primary ] ) ) {
+                        echo "<td class='plugin-title $extra_class'$style><strong>$plugin_name</strong>";
+                        echo $this->row_actions( $actions, true );
+                    } else {
+                        echo "<td class='plugin-title'$style><strong>$plugin_name</strong>";
+                    }
                     echo "</td>";
                     break;
                 case 'description':
-                    echo "<td class='column-description desc'$style>
+                    $classes = 'column-description desc';
+                    if ( $primary === $column_name ) {
+                        $classes .= " $extra_class";
+                    }
+
+                    echo "<td class='$classes'$style>
                         <div class='plugin-description'>$description</div>
                         <div class='$class second plugin-version-author-uri'>";
@@ -633 +645 @@
                     echo implode( ' | ', $plugin_meta );
 
+                    if ( $primary === $column_name ) {
+                        echo $this->row_actions( $actions, true );
+                    }
                     echo "</div></td>";
                     break;
                 default:
-                    echo "<td class='$column_name column-$column_name'$style>";
+                    $classes = "$column_name column-$column_name$class";
+                    if ( $primary === $column_name ) {
+                        $classes .= " $extra_class";
+                    }
+
+                    echo "<td class='$classes'$style>";
 
                     /**
@@ -648 +668 @@
                      */
                     do_action( 'manage_plugins_custom_column', $column_name, $plugin_file, $plugin_data );
+
+                    if ( $primary === $column_name ) {
+                        echo $this->row_actions( $actions, true );
+                    }
                     echo "</td>";
             }
@@ -683 +707 @@
         do_action( "after_plugin_row_$plugin_file", $plugin_file, $plugin_data, $status );
     }
+
+    /**
+     * Get name of default primary column for this specific list table.
+     *
+     * @since 4.3.0
+     * @access protected
+     *
+     * @return string
+     */
+    protected function get_default_primary_column_name() {
+        return 'plugin';
+    }
 }