search

Seravo / wordpress

The WordPress project layout used by many of Seravo's customers, suitable also for local development with Vagrant and git deployment
https://seravo.com
GNU General Public License v3.0
102 stars 54 forks source link

Ship a favicon by default? #150

Open ottok opened 3 years ago

ottok commented 3 years ago

Currently WordPress itself nor the project template serves anything to /favicon.ico requests. This leads to some warnings and errors on sites by default:

image

image

image

This is not a big issue, but many like to run gtmetrix, webpagetest.org, pingdom tools etc and because of this, they will get some negative results by default without always realizing why. To prevent that, the project template could ship a simple default tile as htdocs/favicon.ico.

Comments?

ottok commented 3 years ago

Should we maybe add a WordPress favicon to the project by default to avoid these errors?

Or if we update the default theme to a newer one, would that contain a favicon by itself?