Can't add custom columns to the plugins admin page

[scribu]

The Plugins list table is different from all the rest, in the sense that it uses more than one row per item (simplified):

<tr>
  <td>Plugin Name</td>
  <td>Description</td>
</tr>

<tr class="second">
  <td>Activate | Delete</td>
  <td>Version 1.0 | By: someone</td>
</tr>

<tr class="plugin-update-tr">
  <td class="plugin-update" colspan="3">There's an update available</td>
</tr>

I assume this approach was chosen to make the action links stay at the bottom, when the description is larger.

Problems:

• plugins can't add custom columns
• columns can't be hidden

Supposing these two problems were fixed, we'd still have to have a weird workaround for the colspan attribute in the update notification row.

I suggest we switch to a single row per item and use CSS to position the elements.

[ocean90]

At the moment we have the problem with the "Show on screen" and "Description" option, see screenshot: ​http://grab.by/7dY4

[WraithKenny]

If someone's going to refactor class-wp-plugins-list-table.php maybe they could also add column sorting.

#14987 might be possible to implement at the same time.

[WraithKenny]

Note: CSS positioning on table-cells doesn't work.

[edward mindreantre]

Selected checkbox fix, description tr td fixed.

[edward mindreantre]

Two things have been fixed:

1. Accessability rules say that (1) each checkbox needs a label and (2) every label should be unique. I've whipped something up that fulfills both criteria.

2. Lower down I've moved the description stuff around. It's all now just one tr per plugin and the description gets hidden when using that there "show on screen" checkbox. The more observant of y'all will notice that the select checkbox column becomes enormous when the description is hidden, so a line or two in the css should solve that problem if my solution is satisfactory.

[ocean90]

edward mindreantre, thx for the patch. 2 things:

Notice: Undefined variable: plugin_name in /wordpress/wp-admin/includes/class-wp-plugins-list-table.php on line 386

Also after a reload or using the navigation, only the title will be hidden again.

[edward mindreantre]

Fixed: Description column is hidden when reloading
Fixed: Namn column width is set to auto in order for the checkbox column to not take up half the table.

That should do it. I have no idea why the checkbox and name columns want 50% each...

Anything else broken?

[ocean90]

Switching to a next site via AJAX page navi hides the title only again.

[edward mindreantre]

I believe the new list-table.js will solve your problem.

Unfortunately it has to run the .hide column() routine thing when switching pages which makes it look sorta weird for a few milliseconds. I haven't found where it sets each column width to 50% per default.

I tried setting the cb-column width to max-width 40px or whatever but it doesn't listen. Not after running .hide() on that column. It seems as if .hide() makes the table recalculate column widths.

Can we has-patch this yet? :)

[scribu]

I so hate it when I don't receive email notifications on tickets I've openened.

[scribu]

Common practice is to have all modified files in a single patch.

[edward mindreantre]

All patches in one file.

[scribu]

(In [16346]) Move network only status check out of foreach. See #15318

[scribu]

(In [16347]) Introduce WP_Plugins_List_table:single_row(). See #15318

[scribu]

(In [16348]) Add 'manage_plugin_custom_column' action and properly hide plugin columns. See #15318

[scribu]

(In [16349]) Rename action to 'manage_plugins_custom_column' to be consistent with other screens. See #15318

[scribu]

Initial commit was [16345]

[scribu]

(In [16350]) Make custom plugin columns hideable. See #15318

[scribu]

Only thing left is to figure out what to do with plugin-update-tr.

[scribu]

ocean90 pointed out that WP_MS_Themes_List_Table needs the same treatment.

[ocean90]

Replying to scribu:

ocean90 pointed out that WP_MS_Themes_List_Table needs the same treatment.

Done with 15318.ms.themes.patch.

[scribu]

(In [16361]) Rework WP_MS_Themes_List_Table column display. Props ocean90. See #15318

[scribu]

(In [16368]) Introduce WP_List_Table::get_column_count() and take care of update notice <tr>. Props ocean90 for initial patch. Fixes #15318