Bug #14219
closed
short icon link should contain the MIME type
0%
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)
Updated by old_murraytony over 20 years ago
Also, is there any reason SHORTCUT ICON is in all caps, can it be just shortcut icon or will this break some compatability somewhere?
Updated by Ingmar Schlecht over 20 years ago
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?
Updated by old_murraytony over 20 years ago
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.