Closed Bug 1593322 Opened 5 years ago Closed 5 years ago

Update Sign up tile on Awesomescreen to avoid awkward line break

Categories

(Firefox for Android Graveyard :: Awesomescreen, defect, P1)

Unspecified
Android
defect

Tracking

(firefox-esr68 verified)

VERIFIED FIXED
Tracking Status
firefox-esr68 --- verified

People

(Reporter: mheubusch, Assigned: petru)

Details

(Whiteboard: [fennec68.3])

Attachments

(3 files)

Sean Martell will upload recommended changes to avoid a line break that puts a link on two lines.

Flags: needinfo?(smartell)
Attached image Spec clarification

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!

Flags: needinfo?(smartell)

The priority flag is not set for this bug.
:st3fan, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(sarentz)
Priority: -- → P1
Assignee: nobody → petru.lingurar
Attached image Updated sign-in.png

The easy solution would be to just allow that row containing the sign-in link to be as wide as the parent.

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:
Attachment #9110768 - Flags: approval-mozilla-esr68?

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.

Attachment #9110768 - Flags: approval-mozilla-esr68? → approval-mozilla-esr68+
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(sarentz)
Resolution: --- → FIXED
Whiteboard: [fennec68.3]
Flags: qe-verify+
QA Whiteboard: [qa-triaged]

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.

Flags: qe-verify+

Confirming the fix as well on 68.3.0 RC.

Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: