Waiting on jgruen to generate this asset, as seen in attached screengrab of wires

jgruen - hey, did we get an update from martell et al about the right icon to use?

We *could* ship without this. demoting.

Ah, looks like the current version of the assets is gif, but almost all the icons in gaia currently seem to be png. Arthur, should I update the icons_sprite*.gif files to be PNG, or just leave them as GIFs?

janx in IRC suggested I ask Carol (chuang@) instead, so, switching that now :-)

Per https://bugzilla.mozilla.org/show_bug.cgi?id=840627#c8, using gif format is better in terms of the starting up performance. Not sure if it is still true now, but we should do a profiling before changing the format.

Hi Jared, The attachment is the new sprite for Settings. I've switched the persona icon to Firefox icon. Thanks!

(In reply to Arthur Chen [:arthurcc] PTO: 5/19~5/21 from comment #6) > Per https://bugzilla.mozilla.org/show_bug.cgi?id=840627#c8, using gif format > is better in terms of the starting up performance. Not sure if it is still > true now, but we should do a profiling before changing the format. Cool, thanks Arthur. I'll just stick with the current setup, then.

Comment on attachment 8425600 [details] Github PR 19448 Hey Arthur - Mind having a look at the updated sprite sheet? I've left it as gif to avoid spending extra time doing perf tests on png vs gif. Thanks! Jared

Comment on attachment 8425600 [details] Github PR 19448 It seems the new icon has been added in bug 947985. Jared, could you check if the current icon is as your expectation? If yes, let's simply close this bug.

Nice! Good catch, Arthur. Flashed master on my Flame, and the icon looks just like it should. Closing.