Use the site icon introduced in 4.3 for the My Sites page

[morganestes]

Now that site icons are available in core, we should be able to use them on the network sites/my sites list.

[DrewAPicture]

Cool idea.

[desrosj]

Adjusts the network sites table to show icons.

[desrosj]

My patch adds a column to the wp-admin/network/sites.php page.

We will need to decide what size we want to include here and adjust the patch, as well as add width and height attributes to the image tag and some styling to the column to prevent it from being too wide.

[johnbillion]

I think this should be shown at favicon size next to the site URL.

[desrosj]

Updated patch with favicon sized icon image

[desrosj]

Refreshed patch.

[desrosj]

The refreshed ticket takes care of desktop, but what should we do on mobile? If we want to show it on mobile as well, some styling adjustments will need to be made.

Also, I set the alt text to the same thing that shows in the URL column. Not sure if there is something like "Site Icon" would be more appropriate.

[desrosj]

Before patch

[desrosj]

After patch

[karmatosed]

My gut says we maybe wouldn't show it on mobile. However, that could be because I'm assuming complication visual wise. Do you have a mockup of what it would look like on mobile?

Just so we're clear the blue background is also images not something your patch adds? I'm judging on screenshots not a patch in this comment @desrosj.

[melchoyce]

@desrosj Hey, want to take another look at this after 4.9.6?

[melchoyce]

@desrosj One other question — how does it look if a site doesn't have a favicon chosen?

[jeremyfelt]

Related: #46657, which adds the site icon to the My Sites admin menu.

This would be nice in tandem with that ticket. Moving to future release for now. We can probably get this in 5.3 though.