Bug #14219
closed
short icon link should contain the MIME type
Added by old_murraytony over 20 years ago.
Updated over 20 years ago.
Description
html is outputed like this:
<link rel="SHORTCUT ICON" href="favicon.ico" />
It should output like this:
<link rel="SHORTCUT ICON" href="favicon.ico" type="image/x-icon" />
This can easily be changed in the file class.tslib_pagegen.php
(issue imported from #M183)
Also, is there any reason SHORTCUT ICON is in all caps, can it be just shortcut icon or will this break some compatability somewhere?
You ask why it's all in caps.
I'd ask: Why not?
Now to the main point of this bug:
You opt for including the MIME type parameter type="image/x-icon".
However, isn't it possible to use PNG images as shortcut icons?
If we set the MIME type to image/x-icon, wouldn't this break compatibility for PNG icons?
You are correct about the mime type parameter. If we have the correct file extension even the dumb the browsers should be able to figure what they are dealing with. I retract the bug, you can close it.
As for the SHORTCUT ICON, it is just that everything around it is in lowercase, so it looks out of place. It is a matter of taste.
Closing bug due to user request.
Also available in: Atom
PDF