Closed
Bug 702538
Opened 13 years ago
Closed 12 years ago
[Mobile] Use 32x32 favicon
Categories
(www.mozilla.org :: General, enhancement)
www.mozilla.org
General
Tracking
(Not tracked)
RESOLVED
FIXED
Future
People
(Reporter: kohei, Assigned: sgarrity)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [betterness] u=user c=design p=1)
Attachments
(2 files)
16x16 vs 32x32
Firefox for Android enlarges 16x16 favicons to 32x32.
Use the 32x32 Firefox favicon to make it look better :)
See the attached screenshot for comparison.
|
Reporter | |
Comment 1•13 years ago
|
||
|
|
Reporter | |
Updated•13 years ago
|
Target Milestone: --- → 4.9
|
|
Reporter | |
Comment 2•13 years ago
|
||
This is not a mobile specific bug though, aims to make the favicon look prettier on mobile.
Severity: normal → enhancement
Summary: Use 32x32 favicon → [Mobile] Use 32x32 favicon
Whiteboard: [betterness]
|
|
Reporter | |
Comment 3•13 years ago
|
||
We have switched to 32x32:
http://mozilla.jp/
http://mozilla.jp/firefox/
http://mozilla.jp/thunderbird/
|
Assignee | |
Comment 4•13 years ago
|
||
We should probably use an ico file with both 16px and 32px sizes, no?
|
||
Updated•13 years ago
|
Component: www.mozilla.org/firefox → www.mozilla.org
|
|
Reporter | |
Comment 6•13 years ago
|
||
This is just a replacement of a file... Is there any reason to wait?
SUMO already uses 32x32 favicon:
http://support.mozilla.org/media/img/favicon.ico
|
||
Updated•13 years ago
|
Assignee: nobody → steven
|
|
Assignee | |
Comment 7•13 years ago
|
||
Here's a pull request for the Firefox pages: https://github.com/mozilla/bedrock/pull/213
Don't have a 32px version of the dino-head favicon though. That said, I understand the dino-head is being phased out?
|
||
Comment 8•13 years ago
|
||
No new dino head favicons, please.
|
|
||
Comment 9•13 years ago
|
||
Can we just use this one? http://support.mozilla.org/media/img/favicon.ico
|
||
Comment 10•13 years ago
|
||
Commits pushed to dev at https://github.com/mozilla/bedrock
https://github.com/mozilla/bedrock/commit/c0c25bc9582ba14daaac4814516f1f87a44ce304
Use 32x32 Firefox favicon (Bug 702538)
https://github.com/mozilla/bedrock/commit/8021679d559448d532f611a0f0a11aa5f7f7c100
Merge pull request #213 from sgarrity/bug-702538-favicon
Use 32x32 Firefox favicon (Bug 702538)
|
|
||
Updated•12 years ago
|
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
|
||
Comment 11•12 years ago
|
||
Unless I'm misunderstanding, I think that any benefit from this in Firefox actually requires us to decode the favicon at a higher resolution, which means that it depends on bug 419588 (an image decoder bug).
Depends on: 419588
|
|
Assignee | |
Comment 12•12 years ago
|
||
|
||
Updated•12 years ago
|
Whiteboard: [betterness] → [betterness] u=user c=design p=1
|
|
Assignee | |
Comment 13•12 years ago
|
||
Pull request: https://github.com/mozilla/bedrock/pull/395
|
|
||
Comment 14•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/bedrock
https://github.com/mozilla/bedrock/commit/34d2d0aa31c6318e97e20c9229d331b482c59156
Use 32x32 Firefox favicon (Bug 702538)
https://github.com/mozilla/bedrock/commit/dd5cc98415d2460ab716797911a516dfaff4023a
new favicon (Bug 702538)
https://github.com/mozilla/bedrock/commit/6734b25c82ed01cab4abd3a8aceff8b22fd3190b
Merge pull request #395 from sgarrity/bug-702538-favicon
Bug 702538 favicon
|
|
Assignee | |
Comment 15•12 years ago
|
||
This is in master and will be in production shortly.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•