932833 - [Settings] 1.3 Visual Refresh of Settings icons

Status: RESOLVED FIXED

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

Description

[Amy Lee [:amylee] UX]

A refresh of some of the settings icons for 1.3.

Comment 1

[Amy Lee [:amylee] UX]

Attachment: Settings_Sprite.zip

Hi Pavel, 

Attached are the updated settings sprites. Please note that a new icon has been added to the sprite (chat bubble - messages). Let me know if you have any questions 

Thanks

Comment 2

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

Attachment: patch for Gaia/master

Comment 3

[Eric Pang [:epang] UX]

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

Redirecting feedback from Amy for this bug since she made the changes to the settings icons.  Amy I'll attached screen caps of the updated icons since I know you won't be able to flash Pavel's patch.  If all good please give the patch Feedback+ so Pavel can ask for review.  Thanks!

Comment 4

[Eric Pang [:epang] UX]

Attachment: Screens of new setting icons

Comment 5

[Amy Lee [:amylee] UX]

Attachment: Settings_Sprite_v2.zip

Hey Pavel, 

I noticed that some of the icons aren't centered. I've adjusted the sprite so the icons are sitting centered to bounding boxes of 30x30px and 45x45px. Hopefully this fixes the alignment issue. Can you please replace the sprites with the ones I've attached? 

Also, the sprite background colour set is to #F4F4F4. This is the background colour for v1.3 build. It's slightly lighter than the old build so I assume that the background colour of settings will be changed to this for the 1.3 visual refresh?

Comment 6

[Amy Lee [:amylee] UX]

Attachment: Settings_Icons_1.3.zip

Hi Pavel, 

Here are the rest of the icons for settings that need to be replaced. Let me know if you have any questions. Thanks!

Comment 7

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

Hey Eric can you help with the latest icons, as we talk?

Comment 8

[Eric Pang [:epang] UX]

Attachment: settings icons.zip

Hi Pavel, I've organized the setting icons into the gaia structure.  Thanks!

Comment 9

[Eric Pang [:epang] UX]

flag me for feedback when ready :) thanks!

Comment 10

[Eric Pang [:epang] UX]

Attachment: images to overwrite.zip

Hi Pavel, I just tested the patch.  A few of the images need to be replaced.  Can you use the images attached to replace the ones currently in the patch?  Thanks!

Comment 11

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

Hey Eric,
Sure, can you check the PR again?

Comment 12

[Eric Pang [:epang] UX]

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

everything looks good now, thanks Pavel!

Comment 13

[Evelyn Hung]

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

looks good to me, thank you. :)

Comment 14

[Evelyn Hung]

merged into gaia master: https://github.com/mozilla-b2g/gaia/commit/568cf51284a14e4e855d01a43b355925b09a8e04

Comment 15

[Amy Lee [:amylee] UX]

Hi Pavel, 

I'm still noticing that there is a slight difference in colour between the sprite background and the settings background (you can see faint box outline around each icon). The background colour should be #f4f4f4

Thanks

Comment 16

[Amy Lee [:amylee] UX]

Hey Pavel, 

I just found the bug for changing the settings background. So please disregard my last message. Thanks!


(In reply to Amy from comment #15)
> Hi Pavel, 
> 
> I'm still noticing that there is a slight difference in colour between the
> sprite background and the settings background (you can see faint box outline
> around each icon). The background colour should be #f4f4f4
> 
> Thanks

Comment 17

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

Yeah :) no problem ... I made the patch for the Bug 941628 :)

Comment 18

[Amy Lee [:amylee] UX]

Attachment: Settings_Sprite.zip

Hey Pavel, 
There's been a request to adjust the size and weight of two of the icons (Device Info and Help).

Can you please replace the sprites? Thanks

Comment 19

[Amy Lee [:amylee] UX]

Attachment: Settings_Sprite_v2.zip

Here are the static and active states of the sprites

Comment 20

[Eric Pang [:epang] UX]

(In reply to Amy from comment #19)
> Created attachment 8340108 [details]
> Settings_Sprite_v2.zip
> 
> Here are the static and active states of the sprites

Hey Amy,
Since this bug is already marked resolved fixed can you open a new followup bug and add it to the google doc?  Make sure to attach the assets for Pavel as to the other bug.  Thanks!

Comment 21

[Amy Lee [:amylee] UX]

(In reply to Eric Pang [:epang] from comment #20)
> (In reply to Amy from comment #19)
> > Created attachment 8340108 [details]
> > Settings_Sprite_v2.zip
> > 
> > Here are the static and active states of the sprites
> 
> Hey Amy,
> Since this bug is already marked resolved fixed can you open a new followup
> bug and add it to the google doc?  Make sure to attach the assets for Pavel
> as to the other bug.  Thanks!

Hi Eric, I've created a follow-up bug 944536.