Closed
Bug 1023545
Opened 10 years ago
Closed 10 years ago
[VsD Refresh] Lockscreen Visual Refresh > Time font is too small
Categories
(Firefox OS Graveyard :: Gaia::System::Lockscreen, defect, P1)
Tracking
(blocking-b2g:-, feature-b2g:-)
People
(Reporter: amylee, Assigned: mnjul)
References
Details
(Whiteboard: ux-tracking, visual design, jian [fxos:media] [p=1])
Attachments
(4 files)
The size of the font should be 77px. When measured to the spec it looks too small. Thanks!
Reporter | ||
Comment 1•10 years ago
|
||
Updated•10 years ago
|
Component: Gaia::Camera → Gaia::System::Lockscreen
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jlu
Assignee | ||
Comment 2•10 years ago
|
||
Assignee | ||
Comment 3•10 years ago
|
||
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)
Assignee | ||
Updated•10 years ago
|
Whiteboard: ux-tracking, visual design, jian [fxos:media] → ux-tracking, visual design, jian [fxos:media] [p=1]
Assignee | ||
Updated•10 years ago
|
blocking-b2g: --- → 2.0?
Reporter | ||
Comment 4•10 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+
Assignee | ||
Comment 5•10 years ago
|
||
Yet another trivial patch for the almighty Tim to review!
Attachment #8438870 -
Flags: review?(timdream)
Assignee | ||
Comment 6•10 years ago
|
||
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.
Updated•10 years ago
|
Attachment #8438870 -
Flags: review?(timdream) → review+
Assignee | ||
Comment 7•10 years ago
|
||
Master: https://github.com/mozilla-b2g/gaia/commit/4965b2ccd61f6faa1294a684800c5d81b7690972
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 8•10 years ago
|
||
PLease request approval-gaia-2.0, on this to get this landed on 2.0 branch
blocking-b2g: 2.0? → -
Reporter | ||
Updated•10 years ago
|
feature-b2g: --- → -
Assignee | ||
Updated•10 years ago
|
Target Milestone: --- → 2.0 S4 (20june)
You need to log in
before you can comment on or make changes to this bug.
Description
•