[wp-trac] [WordPress Trac] #44689: Convert user sites (my-sites.php) to table
WordPress Trac
noreply at wordpress.org
Tue Aug 14 18:00:09 UTC 2018
#44689: Convert user sites (my-sites.php) to table
----------------------------------+----------------------------------------
Reporter: eArtboard | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Networks and Sites | Version: 3.0
Severity: normal | Resolution:
Keywords: has-patch 2nd- | Focuses: administration, multisite
opinion |
----------------------------------+----------------------------------------
Comment (by mnelson4):
@shariqkhan2012
> Initially, I planned to add 'Set as Primary Site' as a row action to
replace the current implementation as a dropdown. However, halfway through
the code, I realized there is a do_action hook
('myblogs_allblogs_options') over there, and plugins may be using that
hook to do execute various codes including displaying other content, which
may not go well if it is added to action row.
Ya, while I'd also prefer a row action of "Make Primary Blog", it's
probably better to preserve backwards compatibility in case any plugins
are using that action.
> Additionally, I also planned to add 'Delete' as a row action
Yeah, probably best to do that on a separate ticket afterwards. Let's just
have this focus on the UI change.
@johnjamesjacoby
> These "tiles" could be cards or some other UI that shows more off
(favicons, site descriptions, time zones, etc...)
That's true. I personally think the list table works well enough though,
prefer the consistency, and am unaware of anyone actually making use of
the tiled layout. The current tiled layout kinda looks half-baked IMO.
I tried this patch and it works fine. When I resize the screen so it's
mobile-sized, I see the "show more" toggle button (see
[[Image(https://drive.google.com/a/eventespresso.com/file/d
/1onRBXeTaUso5RQMzJnWdBu-RNFEHfu7O/view?usp=drivesdk)]]) appears but
doesn't do anything, probably because there are no extra columns being
hidden. If this ticket is merged in, however, we could optionally add more
columns (or plugins could) which would make that useful again.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/44689#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list