[wp-trac] [WordPress Trac] #25408: Ability to specify that a list table column is "primary"

WordPress Trac noreply at wordpress.org
Sat May 23 14:21:51 UTC 2015


#25408: Ability to specify that a list table column is "primary"
-----------------------------------------------+-----------------------
 Reporter:  helen                              |       Owner:  stephdau
     Type:  enhancement                        |      Status:  assigned
 Priority:  normal                             |   Milestone:  4.3
Component:  Administration                     |     Version:
 Severity:  normal                             |  Resolution:
 Keywords:  has-patch needs-testing 4.1-early  |     Focuses:
-----------------------------------------------+-----------------------

Comment (by afercia):

 Tested a bit the patch, thanks very much @stephdau :) Noticed a few things
 worth considering:
 - small typo: for Posts/Pages the default primary column is set to
 "author"
 - wouldn't be better to use `$this->screen->id` as dynamic portion of the
 hook name?
 - doing the above, would we really need to pass `$this->screen->id` as
 additional variable?
 - in /network/users.php the "Sites" column needs some special treatment,
 it needs the "has-row-actions" CSS class (links not revealed on focus see
 #31003 for reference)
 - is it worth it to apply all this new stuff to Link Manager too?
 - consider to update the selector in common.js

--
Ticket URL: <https://core.trac.wordpress.org/ticket/25408#comment:22>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list