Closed Bug 1021276 Opened 5 years ago Closed 5 years ago

The Comms Icons Need Colour Tweaks for Differentiation

Categories

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

x86
macOS
defect
Not set

Tracking

(b2g-v2.0 fixed, b2g-v2.1 fixed)

RESOLVED FIXED
2.1 S1 (1aug)
Tracking Status
b2g-v2.0 --- fixed
b2g-v2.1 --- fixed

People

(Reporter: padamczyk, Assigned: kgrandon)

References

Details

(Whiteboard: [systemsfe])

Attachments

(5 files, 2 obsolete files)

Attached image 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.
Assignee: nobody → pla
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.
Blocks: vertical-home-next
No longer blocks: vertical-homescreen
QA Whiteboard: [VH-FL-blocking-]
Hi Kevin,

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

Thanks again!
Assignee: pla → kgrandon
Attachment #8459132 - Attachment is obsolete: true
Attachment #8459133 - Attachment is obsolete: true
Comment on attachment 8459134 [details] [review]
Pull request - Update contacts app icons

Francisco - could you take a look if you have time? Thanks!
Attachment #8459134 - Flags: review?(francisco)
Etienne - Could you take a look if you have time? Thanks!
Attachment #8459135 - Flags: review?(etienne)
Julien - could you take a look if you have time? Thanks!
Attachment #8459136 - Flags: review?(felash)
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 :)
Attachment #8459136 - Flags: review?(felash) → review+
Attachment #8459135 - Flags: review?(etienne) → review?(anthony)
(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 on attachment 8459134 [details] [review]
Pull request - Update contacts app icons

Thanks Kevin!
Attachment #8459134 - Flags: review?(francisco) → review+
Comment on attachment 8459135 [details] [review]
Pull request - Update dialer app icons

We even have tools/png_recompress.sh to do so.
Attachment #8459135 - Flags: review?(anthony) → review+
Blocks: vertical-homescreen
No longer blocks: vertical-home-next
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.
Attachment #8459134 - Flags: approval-gaia-v2.0?(bbajaj)
Whiteboard: [systemsfe]
Target Milestone: --- → 2.0 S6 (18july)
QA Whiteboard: [VH-FL-blocking-] → [VH-FL-blocking-][VH-FC-blocking-]
Attachment #8459134 - Flags: approval-gaia-v2.0?(bbajaj) → approval-gaia-v2.0+
Needs rebasing for v2.0 uplift.
Flags: needinfo?(kgrandon)
Target Milestone: 2.0 S6 (18july) → 2.1 S1 (1aug)
You need to log in before you can comment on or make changes to this bug.