Update Sign up tile on Awesomescreen to avoid awkward line break
Categories
(Firefox for Android Graveyard :: Awesomescreen, defect, P1)
Tracking
(firefox-esr68 verified)
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | verified |
People
(Reporter: mheubusch, Assigned: petru)
Details
(Whiteboard: [fennec68.3])
Attachments
(3 files)
323.73 KB,
image/png
|
Details | |
47 bytes,
text/x-phabricator-request
|
lizzard
:
approval-mozilla-esr68+
|
Details | Review |
254.89 KB,
image/png
|
Details |
Sean Martell will upload recommended changes to avoid a line break that puts a link on two lines.
Comment 1•5 years ago
|
||
To avoid a line break, please ensure there's no right/left margin affecting the content at the button section of the layout. Would involve ensuring the margins of the above content don't continue to the CTA area.
One other edit would be to left align the blurb of text preceding the button as shown in the spec.
Thanks!
Comment 2•5 years ago
|
||
The priority flag is not set for this bug.
:st3fan, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 3•5 years ago
|
||
Updated•5 years ago
|
Assignee | ||
Comment 4•5 years ago
|
||
The easy solution would be to just allow that row containing the sign-in link to be as wide as the parent.
Assignee | ||
Comment 5•5 years ago
|
||
Comment on attachment 9110768 [details]
Bug 1593322 - Allow the sign-in text to be as wide as the parent; r?AndreiLazar
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Important UX issue for a high priority UI element.
- User impact if declined: UX issue - Possible line break of a hyperlink
- Fix Landed on Version:
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Small change.
- String or UUID changes made by this patch:
Comment 6•5 years ago
|
||
Comment on attachment 9110768 [details]
Bug 1593322 - Allow the sign-in text to be as wide as the parent; r?AndreiLazar
Tweak to layout, OK for uplift for Fennec 68.3.
Comment 7•5 years ago
|
||
bugherder uplift |
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 8•5 years ago
|
||
Verified as fixed on ESR builds 68.3b6, 68.3a1.
Devices: Google Pixel (Android Q), Samsung Galaxy Tab S3 (Android 8.0).
However, for the "Sign up" button, the shadow now seems to be missing.
Updated•3 years ago
|
Description
•