939272 - [Status Bar] Update to icons - Airplane Mode, Messages, Geolocation, Tethering

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect, P1)

Description

[Amy Lee [:amylee] UX]

Hi Pavel,

3 of the status bar icons will be need to be updated to reflect the new icons in settings (airplane mode, geolocation, and messages). Sprite attachment to come. 

Thanks!

Comment 1

[Amy Lee [:amylee] UX]

Attachment: StatusBar_Icons.zip

Hi Pavel, 

Attached is the status bar sprite with the updated icons (Airplane Mode, Messages, Geolocation, and Tethering). Let me know if you have any questions. 

Thanks!

Comment 2

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Hey Amy,
can you provide the new sprites based on Bug 933206 of course it will be better if we wait to land the battery icons bug first.

Thanks :)

Comment 3

[Amy Lee [:amylee] UX]

Hi Pavel, 

I attached the sprites to 933206 as well. Is this what you needed?

Comment 4

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Yep :) thanks Amy

Comment 5

[Amy Lee [:amylee] UX]

Attachment: StatusBar_Sprites v2.zip

Hey Pavel, 

Here are the latest sprites with updated icons and keyboard

Comment 6

[Amy Lee [:amylee] UX]

Attachment: Statusbar_Sprite_Updated.zip

Hi Pavel, 

I've attached the updated sprites with the new icons and revised sprite sizes. Let me know if these work for you.

Comment 7

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Hey Amy,
as we talk on irc we need the new icons in to updated sprites from Bug 933206

Comment 8

[Amy Lee [:amylee] UX]

Hi Pavel, 

I've updated the sprites with the new dimension of of 314x96 (from the sprites you gave me). Is this correct?

Comment 9

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Attachment: patch for Gaia/master

Thanks Amy :) I think this are the right images now :) Eric? What do you think :)

Comment 10

[Eric Pang [:epang] UX]

Comment on attachment 8337431 [details] [review]
patch for Gaia/master

Looks good, ready for review.  Thanks Pavel.

Comment 11

[Etienne Segonzac (:etienne)]

Comment on attachment 8337431 [details] [review]
patch for Gaia/master

Forwarding to Tim since I have no idea what to look for here :)

Comment 12

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8337431 [details] [review]
patch for Gaia/master

Thanks!

Comment 13

[Tim Guan-tin Chien [:timdream] (please needinfo)]

master: https://github.com/mozilla-b2g/gaia/commit/93e005f33d11457a6764eb446f0f193fca46c88a