[Form Autofill] The text in autocomplete footer wraps incorrectly

VERIFIED FIXED in Firefox 58

Status

()

defect
P3
normal
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: lchang, Assigned: ralin)

Tracking

(Blocks 1 bug)

unspecified
mozilla59
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 unaffected, firefox58 verified, firefox59 verified)

Details

(Whiteboard: [form autofill:MVP])

Attachments

(3 attachments)

Posted image screenshot.png
Please see the attachment. It only happens when the width of the dropdown is 151px or 152px.

It's a regression cased by bug 1409250.
One liner fix use text-align: center;

Maybe not a regression, but for I should have considered that since flexbox couldn't handle inline stuff very well.
After an offline discussion with Ray, and UX. We prefer to make the footer text shorter string " Preference "  when the width of the dropdown is shorter. And keep the text center aline in all languages. But still, prefer at lease keep English version in one line would be great. : )
Per offline discussion with Mark, we concurred with the fact that it's hard to guarantee text in one-line even for English, in terms of system-level font-size, variable font-width, etc. I'd split the variants of string issue off to another bug, and go text-align: center first. Thanks.
Hey Luke, the patch is an one-liner quickfix, could you help me to review it? Thanks.
Assignee: nobody → ralin
Status: NEW → ASSIGNED
Comment on attachment 8928383 [details]
Bug 1415836 - Make form autofill dropdown footer text centered.

https://reviewboard.mozilla.org/r/199590/#review204710
Attachment #8928383 - Flags: review?(lchang) → review+
Pushed by lchang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/28164096fde2
Make form autofill dropdown footer text centered. r=lchang
https://hg.mozilla.org/mozilla-central/rev/28164096fde2
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Comment on attachment 8928383 [details]
Bug 1415836 - Make form autofill dropdown footer text centered.

Approval Request Comment
[Feature/Bug causing the regression]: Form autofill dropdown footer
[User impact if declined]: terrible looking when words break into two lines (see screenshot)
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: yes, locally
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: very low
[Why is the change risky/not risky?]: only one CSS rules added
[String changes made/needed]: none

Thanks :D
Attachment #8928383 - Flags: approval-mozilla-beta?
Hi Brindusa, could you help find someone to verify if this issue was fixed as expected on the latest Nightly build? Thanks!
Flags: needinfo?(brindusa.tot)
Change NI to Gabi.
Flags: needinfo?(brindusa.tot) → needinfo?(gasofie)
Depends on: 1417775
Verified as fixed with 59.0a1 20171116100106 on Windows 10x64, Ubuntu 14.4 and MacOS 10.12.6
Flags: needinfo?(gasofie)
Comment on attachment 8928383 [details]
Bug 1415836 - Make form autofill dropdown footer text centered.

Issue fixed and verified. Beta58+.
Attachment #8928383 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Verified as fixed with 58.0b5 20171120142222 on Windows 10x64, MacOS 10.12.6 and Ubuntu 14.4
You need to log in before you can comment on or make changes to this bug.