[wp-trac] [WordPress Trac] #3426: Should include default favicon

WordPress Trac wp-trac at lists.automattic.com
Mon Sep 21 19:07:33 UTC 2009


#3426: Should include default favicon
-----------------------------+----------------------------------------------
 Reporter:  robertaccettura  |        Owner:  cpoteet 
     Type:  feature request  |       Status:  reopened
 Priority:  normal           |    Milestone:  2.9     
Component:  Performance      |      Version:  2.8     
 Severity:  normal           |   Resolution:          
 Keywords:  needs-patch      |  
-----------------------------+----------------------------------------------

Comment(by azaozz):

 Replying to [comment:31 ryan]:
 > Granted, if we do it like robots.txt we'll still load a lot of WP unless
 we add a short circuit.

 Yes, so it would still run WP twice on each page load. Seems [comment:8
 revmj] and [comment:15 westi] have the right solution: add another rule to
 htaccess to redirect requests for `/favicon.ico` to `wp-
 includes/images/favicon.ico` when the file doesn't exist.

 Of course this will have to go at the bottom of htaccess so if there's
 already a redirect for `favicon`, we don't override it.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/3426#comment:32>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list