Closed
Bug 968454
Opened 10 years ago
Closed 10 years ago
[Settings] Developer settings menu item needs an icon
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: eeejay, Assigned: janx)
References
Details
Attachments
(2 files)
Currently it is using the accessibility icon, which will look strange once bug #952312 lands.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → janx
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•10 years ago
|
||
Moved over from bug #963039.
Attachment #8379092 -
Flags: review?(anthony)
Comment 2•10 years ago
|
||
Comment on attachment 8379092 [details] [review] gaia pull request I'm not a settings peer so forwarding this. A few notes: - have you ran tools/png_recompress.sh on this image? - We should update the @2x version also.
Attachment #8379092 -
Flags: review?(anthony) → review?(arthur.chen)
Assignee | ||
Comment 3•10 years ago
|
||
(In reply to Anthony Ricaud (:rik) from comment #2) > - have you ran tools/png_recompress.sh on this image? I'm not sure this works on Gif images. My pull request doesn't change current sizes, and Gimp is not able to compress better: 12K icons_sprite.gif 12K icons_sprite_active.gif 16K icons_sprite@1.5x.gif 16K icons_sprite_active@1.5x.gif > - We should update the @2x version also. The 2x versions don't seem to be in sync with the 1x and 1.5x versions. Carol didn't send me an updated 2x version, so I suppose it was not necessary?
Flags: needinfo?(chuang)
Comment 4•10 years ago
|
||
Hello Jan, I added the 2x version to the attachment. thanks :)
Flags: needinfo?(chuang)
Comment 5•10 years ago
|
||
Comment on attachment 8379092 [details] [review] gaia pull request r=me with the 2x icons added. Thanks for the patch!
Attachment #8379092 -
Flags: review?(arthur.chen) → review+
Assignee | ||
Comment 6•10 years ago
|
||
Thanks all! I'm not sure the @2x icon is actually used anywhere, because it wasn't updated for a while.
Comment 7•10 years ago
|
||
Quick question here, the new files are in GIF format but pretty much all of our Gaia assets are in PNG format. Converting the images to PNG and then compressing them as Anthony has suggested in comment 2 provides a nice reduction in size (the images get roughly 1/3 smaller), could we do that?
Comment 8•10 years ago
|
||
The sprites were changed to GIF format for speedup the startup performance (versus PNG format with alpha channel). Please refer to https://bugzilla.mozilla.org/show_bug.cgi?id=840627 for details.
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•