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)

All
Gonk (Firefox OS)
defect
Not set
normal

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: nobody → janx
Status: NEW → ASSIGNED
Attached file gaia pull request
Moved over from bug #963039.
Attachment #8379092 - Flags: review?(anthony)
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)
(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)
Attached file Settings_Sprite@2.zip
Hello Jan,
I added the 2x version to the attachment.
thanks :)
Flags: needinfo?(chuang)
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+
Thanks all! I'm not sure the @2x icon is actually used anywhere, because it wasn't updated for a while.
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?
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.
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.

Attachment

General

Created:
Updated:
Size: