Opened 2 years ago
Closed 2 years ago
#15558 closed task (blessed) (fixed)
Re-factor network/site-users.php to use a list table
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| 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 (8)
Change History (25)
- Type changed from defect (bug) to task (blessed)
TODO:
- Screen Option - Users per page
- Handle redirection from user-edit.php
- Contextual help
Patches welcome ;)
comment:10
follow-up:
↓ 11
JohnONolan — 2 years 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
westi — 2 years 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. :-(
comment:12
beaulebens — 2 years ago
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
JohnONolan — 2 years ago
@westi - want me to patch? :)
comment:14
in reply to:
↑ 13
westi — 2 years ago
comment:15
westi — 2 years ago
comment:16
JohnONolan — 2 years ago
- Keywords has-patch added
comment:17
ryan — 2 years ago
- Resolution set to fixed
- Status changed from new to closed

work in progress