[Email] Update background and sub header colour

RESOLVED FIXED

Status

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: epang, Assigned: joshua-s)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: ux-tracking, visual design, visual-tracking, jian)

Attachments

(3 attachments)

(Reporter)

Description

5 years ago
Created attachment 8336078 [details]
email-screens.png

The following updates are needed in Email set up and settings

Background: f4f4f4
Sub header Background: e7e7e7
(Assignee)

Updated

5 years ago
Assignee: kyee → joshua-smith
Hardware: x86 → Other
(Assignee)

Comment 1

5 years ago
Created attachment 8339537 [details] [review]
PR on GitHub
Attachment #8339537 - Flags: review?
(Assignee)

Comment 2

5 years ago
Created attachment 8339580 [details]
Email app after patch

I did not include the settings screen because I am not able to set up an email account.  It should work though :)
(Reporter)

Comment 3

5 years ago
(In reply to Joshua Smith [:joshua-s] from comment #2)
> Created attachment 8339580 [details]
> Email app after patch
> 
> I did not include the settings screen because I am not able to set up an
> email account.  It should work though :)

Same comment here about changing the font weight to light for the main header text.  This pretty much goes for all of them :)
Flags: needinfo?(joshua-smith)

Comment 4

5 years ago
Eric, The font is correct on device.   On desktop the fonts render a little heavier.
Flags: needinfo?(joshua-smith)

Updated

5 years ago
Attachment #8339537 - Flags: review? → review?(jrburke)

Updated

5 years ago
Attachment #8339537 - Flags: review?(jrburke) → review+

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.