[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