Closed
Bug 888434
Opened 12 years ago
Closed 12 years ago
Icon in tip should match the visual mocks
Categories
(Firefox Health Report Graveyard :: Web: Health Report, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: abc, Assigned: espressive)
Details
(Whiteboard: [android][qa+])
The Firefox icon currently shown in the tip looks different from what we have in Ian's visual mocks. Can we change it to reflect that?
| Assignee | ||
Comment 1•12 years ago
|
||
(In reply to Arun Balachandran Ganesan [:abc] from comment #0)
> The Firefox icon currently shown in the tip looks different from what we
> have in Ian's visual mocks. Can we change it to reflect that?
So, to lessen the number off request and generally lighten the load on the mobile version I am using an icon font that I putting together from icons here: http://fontello.com/
Do we perhaps have an official icon font for Mozilla based sites and products?
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → schalk.neethling.bugs
Updated•12 years ago
|
Whiteboard: [android]
Updated•12 years ago
|
| Assignee | ||
Comment 2•12 years ago
|
||
Any feedback on comment 1 (https://bugzilla.mozilla.org/show_bug.cgi?id=888434#c1)
Flags: needinfo?(abc)
| Reporter | ||
Comment 3•12 years ago
|
||
None that I know of. Maybe, Ian might know?
Flags: needinfo?(abc) → needinfo?(ibarlow)
Comment 4•12 years ago
|
||
We don't have an icon font that I know of. I can make you a proper image of the firefox icon if you give me the dimensions or format you need for it.
Flags: needinfo?(ibarlow)
| Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Ian Barlow (:ibarlow) from comment #4)
> We don't have an icon font that I know of. I can make you a proper image of
> the firefox icon if you give me the dimensions or format you need for it.
Hey there Ian,
For the icon(s), 40px x 40px should work just fine thanks!
Comment 6•12 years ago
|
||
Boom. Here you go sir http://cl.ly/image/04073g441j1V
| Assignee | ||
Comment 7•12 years ago
|
||
(In reply to Ian Barlow (:ibarlow) from comment #6)
> Boom. Here you go sir http://cl.ly/image/04073g441j1V
Thanks Ian
| Assignee | ||
Comment 8•12 years ago
|
||
Sent pull request: https://github.com/mozilla/fhr-jelly/pull/130
| Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Whiteboard: [android]
Updated•11 years ago
|
Whiteboard: [android] → [android][qa+]
Comment 9•11 years ago
|
||
QA verified on dev - the Firefox icon in the /mobile/ tooltip correctly matches what was provided in comment 6.
Status: RESOLVED → VERIFIED
Updated•7 years ago
|
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•