1091714 - Alarm icon sits up too high in status bar

Status: RESOLVED FIXED

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

Description

[Przemek Abratowski [:przemek] UX]

Attachment: 2014-10-29-09-32-42.png

It looks like the alarm icon is a few pixels too high in the status bar when you compare it to the other status bar icons.

Comment 1

[Eric Pang [:epang] UX]

Attachment: Status Bar Icon Sprites

Hey Can you help update the sprites?  I moved the alarm icon a tiny bit up.  Can you flag me for UI-Review when ready so I can check the spacing?  Thanks!

Comment 2

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

Attachment: patch for Gaia/master

Sure :)

can you ui-r?

Comment 3

[Eric Pang [:epang] UX]

(In reply to Pavel Ivanov [:ivanovpavel][:pivanov] from comment #2)
> Created attachment 8524049 [details] [review]
> patch for Gaia/master
> 
> Sure :)
> 
> can you ui-r?

Hey Pavel, can you send me a screen shot?  I just tested and the icons is in the same position so I'm not sure if it's something to do with the sprites or my flashing.  Thanks!

Comment 4

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

Attachment: Shot from Device

Comment 5

[Eric Pang [:epang] UX]

Attachment: Sprites.zip

let's try this again.. but this time in the right direction!

Comment 6

[Eric Pang [:epang] UX]

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

Looks good now, thanks Pavel!

Comment 7

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

Landed to master:
https://github.com/mozilla-b2g/gaia/commit/dc1a6648e0bde12f75583026d53d2b76d103e1b2