[wp-trac] [WordPress Trac] #47398: Remove wp_favicon_request
WordPress Trac
noreply at wordpress.org
Tue Feb 25 15:26:37 UTC 2020
#47398: Remove wp_favicon_request
-----------------------------------------------+---------------------------
Reporter: jonoaldersonwp | Owner:
| SergeyBiryukov
Type: enhancement | Status: closed
Priority: normal | Milestone: 5.4
Component: Bootstrap/Load | Version:
Severity: normal | Resolution: fixed
Keywords: has-patch seo commit has-dev-note | Focuses:
-----------------------------------------------+---------------------------
Comment (by keepmovingdk):
Replying to @SergeyBiryukov:
Thank you for elaborating and for sharing your thoughts.
From my own perspective I'm proud to use WordPress as a platform, and I do
realize that it's easy for people with even a tiny bit of technical
insight to see that a site has been implemented using WordPress.
But I think that exposing the WP logo as favicon is taking it too far.
Because it becomes *very* visible.
I would be less concerned if it only affected new installations. Because
people then would have a chance to notice the favicon while building the
website - and then finding out how to replace it if they don't like it.
But this change will affect all sites upgraded to WP 5.4 (and in such
situations, the developer and/or site owner might not immediately be
aware).
And yes, I do realize that only a very small number of company websites
don't have a favicon.
BTW, what is the recommended, minimal amount of code to add a hook for
outputting the current blank favicon if there is no site icon? My guess is
that it would be to hook into 'do_faviconico' and redirect to the blank
favicon. Does that sound correct to you?
Thanks again.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/47398#comment:26>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list