865906 - AMO should use HTML5 standard way to provide high res favicons

Status: RESOLVED WONTFIX

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Wesley Johnston (:wesj)]

HTML5 provides a way to support high res favicons using the sizes attribute on link tags:

http://www.w3.org/TR/html5/links.html#attr-link-sizes

We support this in our mobile browsers and should support this on Mozilla sites.

Comment 1

[Wesley Johnston (:wesj)]

Currently in Fennec, we ship a nice looking favicon for AMO which gets overwritten with a crappy one the first time you visit the site.

Comment 2

[Christopher Van Wiemeersch [:cvan]]

(In reply to Wesley Johnston (:wesj) from comment #1)
> Currently in Fennec, we ship a nice looking favicon for AMO which gets
> overwritten with a **** one the first time you visit the site.

Can you point to the source of that? I'll gladly swap out.

Comment 3

[Wesley Johnston (:wesj)]

Here's what we ship: 

http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/resources/drawable-mdpi/bookmarkdefaults_favicon_addons.png

a 64x64 of 128x128 version would look even better on high dpi devices (bug anything really is better than 16x16).

Comment 5

[Christopher Van Wiemeersch [:cvan]]

Yeah, I wasn't able to find a version of this icon larger than 32x32. Which is really unfortunate. I'm fine with just using the 32px icon for now.

Comment 6

[Wil Clouser [:clouserw]]

Thanks for filing this.  Due to resource constraints we are closing bugs which we won't realistically be able to fix.  If you have a patch that applies to this bug please reopen.

For more info see http://micropipes.com/blog/2014/09/24/the-great-add-on-bug-triage/