[wp-trac] [WordPress Trac] #47398: Remove wp_favicon_request
WordPress Trac
noreply at wordpress.org
Tue May 28 11:27:27 UTC 2019
#47398: Remove wp_favicon_request
----------------------------+------------------------------
Reporter: jonoaldersonwp | Owner: SergeyBiryukov
Type: enhancement | Status: reviewing
Priority: normal | Milestone: Awaiting Review
Component: Bootstrap/Load | Version:
Severity: normal | Resolution:
Keywords: has-patch seo | Focuses:
----------------------------+------------------------------
Changes (by joostdevalk):
* keywords: has-patch close seo => has-patch seo
Comment:
@ocean90 I'm fine debating with you over this, I'm not fine with you just
adding a `close` keyword: that's just your opinion and you have no more
and no less authority over this than I do. So adding that like this is
RUDE and I don't appreciate it.
By removing this function, we'd allow plugins to hook into the URL with
the rewrite API, currently they can't because we serve a stupid empty
favicon. Our current solution is ridiculously poor. Even shipping a
default WordPress favicon with every install would be better than this, or
even better: outputting the site icon in the favicon.ico.
I'm happy to discuss other paths to a better solution of this problem.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/47398#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list