Closed Bug 1497443 Opened 6 years ago Closed 6 years ago

Text on custom urls Buttons in about:preferences#home page overflows

Categories

(Firefox :: Settings UI, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 64
Tracking Status
firefox-esr60 --- unaffected
firefox62 --- wontfix
firefox63 --- wontfix
firefox64 --- verified

People

(Reporter: cfogel, Assigned: Gijs)

References

(Blocks 1 open bug)

Details

(Keywords: regression)

Attachments

(2 files)

Attached image as_preff_TextButtonOverflow.png (deleted) —
[Affected versions]: - 62.0.3, 63.0b12, 64.0a1 (2018-10-08) [Affected platforms]: - win 10x64, Ubuntu 16.04, macOS 10.13 [Steps to reproduce]: 1. Launch Firefox; 2. Access about:preferences#home 3. Click the dropdown for the Homepage and new windows option; 4. Click on Custom URLs... 5. Grab the browser and resize to shrink its width. [Expected result]: - content is properly displayed; - not sure of the proper behavior, but probably the button width should remain the same to contain the text; [Actual result]: - the following buttons are changing width to adjust for resize but text is overflowing: [Use Current Pages], [Use Bookmark...] [Regression range]: - introduced with bug 1417155 [Additional notes]: - attached screenshot with the issue;
This is caused by the buttons having -moz-box-flex (flex=1 attribute) and a defined min-width ( https://searchfox.org/mozilla-central/source/toolkit/themes/osx/global/button.css#12 , https://searchfox.org/mozilla-central/source/toolkit/themes/linux/global/button.css#16, https://searchfox.org/mozilla-central/source/toolkit/themes/windows/global/button.css#16 ), which in XUL means they can be resized as part of flex. There's a trivial fix for this, I'll just put up a patch. While this may have regressed as part of bug 1417155, that's only because this stuff is now in its own column so there's less available width in the first place. I bet in locales with longer labels this would have been reproducible in the old layout, prior to bug 1417155.
Assignee: nobody → gijskruitbosch+bugs
Blocks: 1417155
Status: NEW → ASSIGNED
Priority: -- → P1
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/59b646e9e899 set min-width to auto for flexed buttons in the prefs, r=jaws
Blocks: 1500589
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Verified with 64.0a1 (2018-10-21) on macOS_10.14, Ubuntu_16.04, Win_10x64.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: