Closed
Bug 1415836
Opened 5 years ago
Closed 5 years ago
[Form Autofill] The text in autocomplete footer wraps incorrectly
Categories
(Toolkit :: Form Autofill, defect, P3)
Toolkit
Form Autofill
Tracking
()
VERIFIED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox57 | --- | unaffected |
firefox58 | --- | verified |
firefox59 | --- | verified |
People
(Reporter: lchang, Assigned: ralin)
References
(Blocks 1 open bug)
Details
(Whiteboard: [form autofill:MVP])
Attachments
(3 files)
42.53 KB,
image/png
|
Details | |
26.29 KB,
image/png
|
Details | |
59 bytes,
text/x-review-board-request
|
lchang
:
review+
gchang
:
approval-mozilla-beta+
|
Details |
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.
Assignee | ||
Comment 1•5 years ago
|
||
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.
Reporter | ||
Updated•5 years ago
|
status-firefox57:
--- → unaffected
status-firefox58:
--- → affected
Comment 2•5 years ago
|
||
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. : )
Assignee | ||
Comment 3•5 years ago
|
||
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.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 5•5 years ago
|
||
Hey Luke, the patch is an one-liner quickfix, could you help me to review it? Thanks.
Assignee: nobody → ralin
Status: NEW → ASSIGNED
Reporter | ||
Comment 6•5 years ago
|
||
mozreview-review |
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
Comment 8•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/28164096fde2
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Assignee | ||
Comment 9•5 years ago
|
||
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?
Comment 10•5 years ago
|
||
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)
Comment 12•5 years ago
|
||
Verified as fixed with 59.0a1 20171116100106 on Windows 10x64, Ubuntu 14.4 and MacOS 10.12.6
Flags: needinfo?(gasofie)
Comment 13•5 years ago
|
||
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+
Comment 14•5 years ago
|
||
bugherderuplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/c78d02a2b591
Comment 15•5 years ago
|
||
Verified as fixed with 58.0b5 20171120142222 on Windows 10x64, MacOS 10.12.6 and Ubuntu 14.4
Updated•5 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•