The current module only allows .ico files using mimetypes similar to */ico. It would great to support additional mimetypes such as image/png.

Comments

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented

So where did you find one with image/png?

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Status: Active » Closed (won't fix)

The current implementation is based on details at [1].
Won't fix for no reason.

[1] http://www.iana.org/assignments/media-types/image/vnd.microsoft.icon

jonathan_hunt’s picture

jonathan_hunt commented
Status: Closed (won't fix) » Active

My apologies for the lack of earlier response.
PNG is recommended by W3C: http://www.w3.org/2005/10/howto-favicon

I'll see if I can track down some examples.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Assigned: Unassigned » sanduhrs
Status: Active » Fixed

That is sufficient.

Commited to 5.x, 6-dev and HEAD.
Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.