search

mattybeard / Destiny-Daily-2

Code for the Destiny Daily website, a Destiny helper tool - this project has now been retired but the code left up or legacy reasons
1 stars 0 forks source link

Remove duplicate favicon #13

Closed benedfit closed 7 years ago

benedfit commented 8 years ago

There is an old favicon in the root of the site. Move the one from /assets/img/meta to the root and then update the path in <head> of Layout

mattybeard commented 8 years ago

Favicon removed from route. @benedfit can we not just leave as is now?

benedfit commented 8 years ago

It's actually better for us to move it, lots of old browsers auto check for /favicon.ico, so it prevents lots of 404s just to have it in the root. I need to update the meta icons, as I missed a few, anyway, so I'll make the change when I do that

mattybeard commented 8 years ago

Moved in live code, just need to move in template code for next update.