Ticket #15558 (closed task (blessed): fixed)

Opened 18 months ago

Last modified 18 months ago

Re-factor network/site-users.php to use a list table

Reported by: PeteMall Owned by: PeteMall
Priority: normal Milestone: 3.1
Component: Multisite Version: 3.1
Severity: normal Keywords: has-patch
Cc:

Description

Adapt wp-admin/includes/class-wp-users-list-table.php for the Network Admin Site Users page.

Attachments

15558.diff Download (7.9 KB) - added by PeteMall 18 months ago.
work in progress
15558.2.diff Download (12.2 KB) - added by PeteMall 18 months ago.
15558.3.diff Download (12.1 KB) - added by PeteMall 18 months ago.
Updated cap checks.
15558.screen-options.diff Download (1.0 KB) - added by duck_ 18 months ago.
15558.hide.posts.col.patch Download (932 bytes) - added by ocean90 18 months ago.
15558.4.diff Download (4.5 KB) - added by PeteMall 18 months ago.
Update and error messages.
15558.5.diff Download (5.2 KB) - added by PeteMall 18 months ago.
Create new user form.
15558.6.diff Download (606 bytes) - added by JohnONolan 18 months ago.
Amends CSS to be global classes not associated with specific tags.

Change History

work in progress

Updated cap checks.

(In [16560]) Adapt user-list-table for network admin site-users. See #15558

  • Type changed from defect (bug) to task (blessed)

TODO:

  • Screen Option - Users per page
  • Handle redirection from user-edit.php
  • Contextual help

Patches welcome ;)

duck_18 months ago

(In [16570]) Add screen option in site-users. Props duck_, see #15558

(In [16573]) Hide posts column for site-users. Props ocean90, see #15558

Update and error messages.

Latest patch looks like a good step in the right direction

(In [16582]) Update and error messages for site-users. See #15558.

(In [16584]) Exit after wp_redirect(). See #15558

Create new user form.

(In [16586]) Create new user form for site-users. See #15558

(In [16607]) Make the list of actions that can be performed on a site in the User List in Network admin filterable. Add classes for all core site status types to the view url. Move the CSS to a css file rather than including it inline. See #15558

comment:10 follow-up: ↓ 11   JohnONolan18 months ago

Suggestion:

Instead of changing

tr.site-mature {

to

tr.site-mature, a.site-mature {

Why don't we change it to

.site-mature {

Unless there are other instances of .site-mature within different tags which require different styling then this would be the best way to go. Same for .site-deleted and .site-spammed and .site-archived

comment:11 in reply to: ↑ 10   westi18 months ago

Replying to JohnONolan:

Suggestion:

Instead of changing

tr.site-mature {

to

tr.site-mature, a.site-mature {

Why don't we change it to

.site-mature {

Unless there are other instances of .site-mature within different tags which require different styling then this would be the best way to go. Same for .site-deleted and .site-spammed and .site-archived

Probably true - I was too lazy to search for other instances at the time I was doing this. :-(

User rows are not currently highlighted when a user is set as spam. Changing line 146 to this will fix it:

$alt .= " $col"; 

comment:13 follow-up: ↓ 14   JohnONolan18 months ago

@westi - want me to patch? :)

comment:14 in reply to: ↑ 13   westi18 months ago

Replying to JohnONolan:

@westi - want me to patch? :)

Please

(In [16879]) Fix row hilighting when user is spam. See #15558 props beaulebens.

Amends CSS to be global classes not associated with specific tags.

  • Keywords has-patch added
  • Status changed from new to closed
  • Resolution set to fixed

(In [16902]) Make classes global. Props JohnONolan. fixes #15558

Note: See TracTickets for help on using tickets.