1021276 - The Comms Icons Need Colour Tweaks for Differentiation

Status: RESOLVED FIXED

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

Description

[Patryk Adamczyk [:patryk] UX]

Attachment: First row of icons

Currently the first 3 icons are all the same colour and its hard to see the difference.
Perhaps Phone needs to be more green.
SMS needs to be more blue.

Comment 1

[Kevin Grandon :kgrandon]

When will we have these? I'm not sure what kind of blocking criteria we will have for 2.0, so we would ideally have these assets today if we want to include them.

Comment 2

[Peter La]

Attachment: FFOS2CommsAppIconUpdate_July18.zip

Hi Kevin,

Here are the new Comms icons with greater separation of colour.  I'm going to assign it to you now.

Thanks again!

Comment 3

[Kevin Grandon :kgrandon]

Attachment: https://github.com/mozilla-b2g/gaia/pull/21950/files

Comment 4

[Kevin Grandon :kgrandon]

Attachment: Pull request - Update contacts app icons

Comment 5

[Kevin Grandon :kgrandon]

Attachment: Pull request - Update contacts app icons

Comment 6

[Kevin Grandon :kgrandon]

Comment on attachment 8459134 [details] [review]
Pull request - Update contacts app icons

Francisco - could you take a look if you have time? Thanks!

Comment 7

[Kevin Grandon :kgrandon]

Attachment: Pull request - Update dialer app icons

Etienne - Could you take a look if you have time? Thanks!

Comment 8

[Kevin Grandon :kgrandon]

Attachment: Pull request - Update messages app icons

Julien - could you take a look if you have time? Thanks!

Comment 9

[Julien Wajsberg [:julienw]]

Comment on attachment 8459136 [details] [review]
Pull request - Update messages app icons

r=me but you need to optimize the png files before merging. optipng (which I think is even not the best) could decrease the file size from 15% to 20% depending on the file, this is good to take IMO :)

Comment 10

[Kevin Grandon :kgrandon]

(In reply to Julien Wajsberg [:julienw] from comment #9)
> Comment on attachment 8459136 [details] [review]
> Pull request - Update messages app icons
> 
> r=me but you need to optimize the png files before merging. optipng (which I
> think is even not the best) could decrease the file size from 15% to 20%
> depending on the file, this is good to take IMO :)

Good point, forgot about this. Will update these patches.

Comment 11

[Francisco Jordano [:arcturus] [:francisco]]

Comment on attachment 8459134 [details] [review]
Pull request - Update contacts app icons

Thanks Kevin!

Comment 12

[Anthony Ricaud (:rik)]

Comment on attachment 8459135 [details] [review]
Pull request - Update dialer app icons

We even have tools/png_recompress.sh to do so.

Comment 13

[Kevin Grandon :kgrandon]

Thanks all.

Contacts: https://github.com/mozilla-b2g/gaia/commit/d30ad1662540da85e9013e5a7242761ffb09c8f1
Dialer: https://github.com/mozilla-b2g/gaia/commit/7bfb34f67eceae1878288f18b7ff9601013627e0
Messages: https://github.com/mozilla-b2g/gaia/commit/628490a9c482c1034f173011ffeb6170a0b9d11b

Comment 14

[Kevin Grandon :kgrandon]

Comment on attachment 8459134 [details] [review]
Pull request - Update contacts app icons

Would like to request approval for 2.0 with the patches from these three bugs. All three patches are icons only and extremely low risk which give us some additional visual polish for the vertical homescreen.

Comment 15

[Ryan VanderMeulen [:RyanVM]]

Needs rebasing for v2.0 uplift.

Comment 16

[Kevin Grandon :kgrandon]

Squashed and uplifted: https://github.com/mozilla-b2g/gaia/commit/1bca63303bada6b53c0e50bc466c2a7d9d0a27a7