search

xeoncross / jr

Jr. the static, static javascript site generator (you should see this)
Other
759 stars 39 forks source link

Added support for favicon.ico #14

Closed mr-ex closed 10 years ago

mr-ex commented 10 years ago

I just sort of added this for myself, there should probably be better support for disabling it with a null value...

xeoncross commented 10 years ago

This isn't actually needed. By default browsers request example.com/favicon.ico from the root folder. So, just drop your favicon.ico right next to the index.html and you are good to go!

mr-ex commented 10 years ago

The rel method is characterized as preferred by the W3C[1], but you're right that many server/browser configurations will work with the discouraged[1] method of picking up a file named favicon.ico in the root directory.

[1] - http://www.w3.org/2005/10/howto-favicon