Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #54447


Ignore:
Timestamp:
12/14/2021 06:15:42 PM (2 years ago)
Author:
sabernhardt
Comment:

Hi and welcome to Trac!

Thanks for the report. An early proposal on #46657 used a filter, and it still could have one to check whether the site icons should be included. Perhaps the filter could be something like this:

if ( apply_filters( 'show_site_icons_in_toolbar', true ) && has_site_icon() ) {

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #54447

    • Property Keywords needs-patch added
    • Property Component changed from Administration to Toolbar
    • Property Type changed from defect (bug) to enhancement
    • Property Milestone changed from Awaiting Review to Future Release
  • Ticket #54447 – Description

    initial v1  
    11WordPress 5.8 added displaying the site icon on the My Sites menu. When a site has an icon, it takes 0.05 seconds per get_site_icon_url, and there are three of them per site, for a total of 0.15 seconds per site. When a multisite has hundreds of sites and 100 hundred of them have site icons, this adds up and adds 15 seconds to the page load time.
    22
    3 The change causing performance issues: https://core.trac.wordpress.org/changeset/50834
     3The change causing performance issues: r50834