939152 - [Action Icons - Shared] Remove shadows from action icons

Status: RESOLVED FIXED

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

Description

[Amy Lee [:amylee] UX]

Hi Pavel,

Please replace the action icons in this app with the ones that I'll be providing (I'll be attaching new icons to this bug). This is a blocker for Bug 939106. 

Thanks!

Comment 1

[Eric Pang [:epang] UX]

Attachment: shared.zip

Hi Pavel,

Here are some shared icons i found that need to be updated, I've replaced them within the folder structure
gaia/shared...

Thanks!

Comment 2

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

Attachment: patch for Gaia/master

Comment 3

[Eric Pang [:epang] UX]

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

Looks good, ready for review. Thanks!

Comment 4

[Eric Pang [:epang] UX]

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

Hi Vivien, can you help review this? Thanks!

Comment 5

[Fabien Cazenave [:kaze]]

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

Please re-ping Vivien for the review if my r+ is not enough.

Comment 6

[Eric Pang [:epang] UX]

(In reply to Fabien Cazenave [:kaze] from comment #5)
> Comment on attachment 8333489 [details] [review]
> patch for Gaia/master
> 
> Please re-ping Vivien for the review if my r+ is not enough.

Thanks Kaze!

Comment 7

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

Thanks again :)

Landed in master:
https://github.com/mozilla-b2g/gaia/commit/2fead1c985f28b46b122ca17b0a98be6cf7beaad