Loading...
 
Skip to main content

Favicon not correctly implemented.

  • Tracker item actions
List all trackers List items
Status
Closed
Subject
Favicon not correctly implemented.
Version
1.9.x
Category
  • Error
  • Usability
Feature
Meta Tag
Theme: Look & feel, Styles, CSS, Theme Control Center
Resolution status
Out of Date
Submitted by
SeanBDurkin
Lastmod by
drsassafras
Rating
(0)
Description

The implementation of favicons is incorrect for types other than .png.

Tiki currently renders the favicon as:

Current rendering
Copy to clipboard
<link rel="icon" href="favicon.png" />


Notice that there is no declaration of MIME type (very bad). If you goto Administration | Look & feel | Other, there is a pull-down box to select the favicon icon MIME type, which gives choices:

  • image/png
  • image/bmp (wrong!)
  • image/x-icon (wrong!)


None of these selections actually effect the LINK tag! In other words, someone forgot to implement the MIME attribute of the link tag. Even if it was implemented, these choices would be wrong (see solution).

Also on the Look & feel | Other page, there is a 32 character limit on the favicon icon file name. This limit is too small and is problematic for sites where the favicon is not located in the tiki root directory.

Solution

(1) Correctly implement favicons. Here are 3 excellent resources for that purpose:


For standard browsers the correct rendoring should be either:

Copy to clipboard
<link rel="icon" type="image/vnd.microsoft.icon" href="http://example.com/image.ico"> <link rel="icon" type="image/png" href="http://example.com/image.png"> <link rel="icon" type="image/gif" href="http://example.com/image.gif"> <link rel="icon" type="image/jpeg" href="http://example.com/image.jpg">


For Internet Explorer, versions 5 to 7

Copy to clipboard
<link rel="SHORTCUT ICON" href="http://http://example.com/image.ico"/>


I dont know about IE support for non-.ico type favicons.

(2) Menu choices should be updated to:

  • Auto (MIME type guessed from file extension)
  • image/vnd.microsoft.icon
  • image/png
  • image/gif
  • image/jpeg


(3) The character limit for favicon URL (it should be called URL not file-name), should be extended to something like 132 characters.

Workaround
According to the wiki page, it looks like the current implementation is perhaps overkill now, as specifying a favicon with the link tag is no longer required.
Importance
5
Priority
25
Demonstrate Bug on Tiki 19+
Please demonstrate your bug on show2.tiki.org
Demonstrate Bug (older Tiki versions)
Ticket ID
2728
Created
Sunday 06 September, 2009 05:55:40 UTC
by Unknown
LastModif
Thursday 17 November, 2016 04:35:29 UTC


Collapse/expand modules below
Show PHP error messages