Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

favicon adjustment #59

Closed
russmaxdesign opened this issue May 9, 2014 · 1 comment
Closed

favicon adjustment #59

russmaxdesign opened this issue May 9, 2014 · 1 comment
Labels

Comments

@russmaxdesign
Copy link

I noticed that you use the following for favicon link:

<link rel="shortcut icon" type="image/ico" href="img/favicon.ico">

I have been doing extensive reading on this recently and found that:

  1. We do NOT need the type="image/ico" for older versions of IE:
  2. We DO need to use the invalid "shortcut icon" as opposed to "icon" if you want to support IE versions earlier than 11 (how insane is that!)
  3. We must either place in the ROOT or place it in another location and then link to this favicon using a FULLY QUALIFIED URL (ie. contains the http:// or https://www).

More here:
http://msdn.microsoft.com/en-us/library/ms537656(VS.85).aspx
http://en.wikipedia.org/wiki/Favicon
http://jeffcode.blogspot.com.au/2007/12/why-doesnt-favicon-for-my-site-appear.html

So, your code may need to change to either placing the favicon in the root, without a full URL:

<link rel="shortcut icon" href="/favicon.ico">

or, place in the "img" folder, but include a fully qualified URL

<link rel="shortcut icon" href="http://sitername.com/img/favicon.ico">

Also, you can include more options for IE10 metro tiles... if you want :)

<meta name="msapplication-square150x150logo" content="images/logo.png">
<meta name="msapplication-square310x310logo" content="images/largelogo.png">
<meta name="msapplication-square70x70logo" content="images/tinylogo.png">
<meta name="msapplication-wide310x150logo" content="images/widelogo.png">

More here:
http://msdn.microsoft.com/en-us/library/ie/dn255024(v=vs.85).aspx

HTH
Russ

@cferdinandi cferdinandi added the bug label Jun 4, 2014
@cferdinandi
Copy link
Owner

Fixed in v3.6.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants