If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[VsD Refresh] Lockscreen Visual Refresh > Time font is too small

RESOLVED FIXED in 2.0 S4 (20june)

Status

Firefox OS
Gaia::System::Lockscreen
P1
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: amylee, Assigned: mnjul)

Tracking

unspecified
2.0 S4 (20june)
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(blocking-b2g:-, feature-b2g:-)

Details

(Whiteboard: ux-tracking, visual design, jian [fxos:media] [p=1])

Attachments

(4 attachments)

(Reporter)

Description

3 years ago
The size of the font should be 77px. When measured to the spec it looks too small.

Thanks!
(Reporter)

Comment 1

3 years ago
Created attachment 8437952 [details]
Fontsize_Lockscreen_Time.png
Component: Gaia::Camera → Gaia::System::Lockscreen
Assignee: nobody → jlu
Created attachment 8438177 [details]
lockscreen_time_7.7rem.png
Created attachment 8438181 [details]
lockscreen_time_8.5rem.png

Hi Amy,

Thanks for your comments. While the CSS rule is already specifying 77px (actually it's 7.7rem to account for device pixel ratio), your concern is fully understood.

I mocked up a screenshot with 85px (8.5rem) text size. How do you think?
Note it's normal that 8.5rem feels a bit "fatter" on the phone than the spec since we don't have Ultra Light assets on the phone (yet). Please check the attachments. Much appreciated!
Attachment #8438181 - Flags: ui-review?(amlee)
Whiteboard: ux-tracking, visual design, jian [fxos:media] → ux-tracking, visual design, jian [fxos:media] [p=1]
blocking-b2g: --- → 2.0?
(Reporter)

Comment 4

3 years ago
Comment on attachment 8438181 [details]
lockscreen_time_8.5rem.png

Hi John, 

The font size looks good now. Do you know when we will have the ultra light weight of the font? Thanks
Attachment #8438181 - Flags: ui-review?(amlee) → ui-review+
Created attachment 8438870 [details]
Patch

Yet another trivial patch for the almighty Tim to review!
Attachment #8438870 - Flags: review?(timdream)
Depends on: 987872
Hi Amy,

From what I can see, bug 987872 is still actively discussing about the possibility of landing the new fonts into master/2.0 (apparently it looks quite probable). I will keep an eye on it and see if we need to respond accordingly.
Attachment #8438870 - Flags: review?(timdream) → review+
Master: https://github.com/mozilla-b2g/gaia/commit/4965b2ccd61f6faa1294a684800c5d81b7690972
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
PLease request approval-gaia-2.0, on this to get this landed on 2.0 branch
blocking-b2g: 2.0? → -
Blocks: 1024845
No longer blocks: 1024845
(Reporter)

Updated

3 years ago
feature-b2g: --- → -
Target Milestone: --- → 2.0 S4 (20june)
You need to log in before you can comment on or make changes to this bug.