Closed Bug 1405781 Opened 2 years ago Closed 2 years ago

Customize overflow panel scrollbar extends past edge

Categories

(Firefox :: Toolbars and Customization, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 58
Tracking Status
firefox57 --- verified
firefox58 --- verified

People

(Reporter: osmose, Assigned: Gijs)

Details

(Whiteboard: [reserve-photon-structure])

Attachments

(2 files)

In customize mode on MacOS, if the window is short enough that the overflow panel overflows it's content, the scrollbar on the right side sticks out past the edge of the panel. The attached screenshot shows the bug with the patches from bug 1405643 applied to fix other overflow issues, but the scrollbar extends past the edge with or without those patches.
Whiteboard: [photon-structure][triage]
Flags: qe-verify?
Priority: -- → P4
Whiteboard: [photon-structure][triage] → [reserve-photon-structure]
Hrmpf, this also affects windows 10. Probably need to look at this soon-ish, after all. :-(
Flags: qe-verify? → qe-verify+
OS: Mac OS X → All
Priority: P4 → P3
QA Contact: gwimberly
Hardware: Unspecified → All
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Priority: P3 → P1
Comment on attachment 8916993 [details]
Bug 1405781 - don't constrain the width of the overflow panel in customize mode to avoid the scrollbar being placed outside it,

https://reviewboard.mozilla.org/r/188032/#review193194

::: browser/themes/shared/customizableui/panelUI.inc.css:1741
(Diff revision 1)
>  }
>  
> +/* In customize mode, the overflow list is constrained by its container,
> + * so we set width: auto to avoid the scrollbar not fitting.
> + */
> +#customization-panelHolder .widget-overflow-list {

Self-nit: this should be a child selector
Comment on attachment 8916993 [details]
Bug 1405781 - don't constrain the width of the overflow panel in customize mode to avoid the scrollbar being placed outside it,

https://reviewboard.mozilla.org/r/188032/#review193198

::: browser/themes/shared/customizableui/panelUI.inc.css:1741
(Diff revision 1)
>  }
>  
> +/* In customize mode, the overflow list is constrained by its container,
> + * so we set width: auto to avoid the scrollbar not fitting.
> + */
> +#customization-panelHolder .widget-overflow-list {

Well, that'd be the only remark I'd have as well. :-D
Attachment #8916993 - Flags: review?(mdeboer) → review+
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/26b13cdf8add
don't constrain the width of the overflow panel in customize mode to avoid the scrollbar being placed outside it, r=mikedeboer
https://hg.mozilla.org/mozilla-central/rev/26b13cdf8add
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Comment on attachment 8916993 [details]
Bug 1405781 - don't constrain the width of the overflow panel in customize mode to avoid the scrollbar being placed outside it,

Approval Request Comment
[Feature/Bug causing the regression]:
[User impact if declined]: Please see a screenshot of the result in attachment 8915243 [details].
[Is this code covered by automated tests?]: No.
[Has the fix been verified in Nightly?]: No.
[Needs manual test from QE? If yes, steps to reproduce]: Yes. STR:
1. Open Customize Mode (available from the App Menu (aka. Hamburger menu)
2. Drag a number of widgets into the Overflow Menu area, like 5.
3. Resize the browser window small enough to have scrollbars appear on the Overflow Menu area
4. Without this patch the scrollbar should be positioned slightly offset and with the patch it should fit neatly into the area.
[List of other uplifts needed for the feature/fix]: n/a.
[Is the change risky?]: No.
[Why is the change risky/not risky?]: This is a very small graphical CSS change that is constrained to Customize Mode only, not other areas of the browser.
[String changes made/needed]: n/a.
Attachment #8916993 - Flags: approval-mozilla-beta?
Comment on attachment 8916993 [details]
Bug 1405781 - don't constrain the width of the overflow panel in customize mode to avoid the scrollbar being placed outside it,

Photon polish, Beta57+
Attachment #8916993 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Reproduced this issue using STR from comment 8, on a Nightly build (2017-10-04).

Not reproducible anymore on Win 10 x64, Mac OS X 10.11 and Ubuntu 16.04 x64 with latest Nightly 58.0a1 (20171011220113). I will verify this in 57 Beta 8 as well.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Verified fixed on 57 beta 12 as well across platforms: Win 10 x64, Ubuntu 16.04 x64 and macOS 10.13.
You need to log in before you can comment on or make changes to this bug.