No favicon / default favicon image is very fuzzy

VERIFIED FIXED in Firefox 11

Status

()

Firefox for Android
General
P4
normal
VERIFIED FIXED
6 years ago
a year ago

People

(Reporter: patryk, Assigned: mbrubeck)

Tracking

11 Branch
Firefox 12
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, firefox12 verified)

Details

Attachments

(3 attachments)

(Reporter)

Description

6 years ago
Created attachment 585850 [details]
Fuzzy icon example

Higher quality / multiple image sizes of the no favicon / default favicon are needed.
Assignee: nobody → padamczyk
Priority: -- → P4
(Reporter)

Comment 1

6 years ago
Created attachment 588055 [details]
Update default favicons (5 sizes)
(Reporter)

Comment 2

6 years ago
Attached are the production assets.
Assignee: padamczyk → nobody
Assignee: nobody → mbrubeck
(Assignee)

Comment 3

6 years ago
Created attachment 591603 [details] [diff] [review]
patch

Sorry for taking a while to get to this, Patryk.  This bug adds new default favicon images at the following sizes as used in the awesomescreen:

MDPI:  32px
HDPI:  48px
XHDPI: 64px

The favicon in the URL bar is drawn at a smaller size (21.33dp, which maps to the pixel sizes below).  Currently it is just a scaled-down version of the default favicon.  If you'd like to provide a separate image for the smaller URL bar icon, it should be at the following sizes:

MDPI:  21px
HDPI:  32px (we already have an icon available at this size)
XHDPI: 43px (I think - can someone with an XHDPI device test this?)
Attachment #591603 - Flags: review?(sriram)
Comment on attachment 591603 [details] [diff] [review]
patch

Review of attachment 591603 [details] [diff] [review]:
-----------------------------------------------------------------

The patch looks fine to me.
I haven't tested it yet though.
Attachment #591603 - Flags: review?(sriram) → review+
(Assignee)

Comment 5

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/8113ce367f8f
Status: NEW → ASSIGNED
status-firefox11: --- → affected
status-firefox12: --- → fixed
Target Milestone: --- → Firefox 12
(Assignee)

Comment 6

6 years ago
Comment on attachment 591603 [details] [diff] [review]
patch

[Approval Request Comment]
Trivial Android-only change to tweak some image files.  Fixes some fuzzily-scaled images in the UI.
Attachment #591603 - Flags: approval-mozilla-aurora?

Comment 7

6 years ago
Comment on attachment 591603 [details] [diff] [review]
patch

[Triage Comment]
Mobile only - approved for Aurora.
Attachment #591603 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Comment 8

6 years ago
https://hg.mozilla.org/mozilla-central/rev/8113ce367f8f
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
This. Is. Lovely.

Verified Fixed on M-C
Samsung Galaxy Nexus (Android 4.0.3)
20120126031113
http://hg.mozilla.org/mozilla-central/rev/402b394b6623

Aurora nom?
Status: RESOLVED → VERIFIED
status-firefox12: fixed → verified
OS: Mac OS X → Android
Hardware: x86 → ARM
(Assignee)

Comment 10

6 years ago
https://hg.mozilla.org/releases/mozilla-aurora/rev/0cc55c3cff25
status-firefox11: affected → fixed
You need to log in before you can comment on or make changes to this bug.