Closed Bug 2010864 Opened 3 months ago Closed 2 months ago

Applications action menu dropdown option overflows and causes scroll

Categories

(Firefox :: Settings UI, defect, P3)

defect

Tracking

()

VERIFIED FIXED
149 Branch
Tracking Status
firefox-esr115 --- unaffected
firefox-esr140 --- unaffected
firefox147 --- unaffected
firefox148 --- unaffected
firefox149 --- disabled
firefox150 --- verified

People

(Reporter: mkennedy, Assigned: tgiles)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [recomp])

Attachments

(4 files, 2 obsolete files)

Steps to reproduce

  1. Go to Settings
  2. Scroll down to Applications section
  3. Click to open last dropdown menu in Actions column

Actual Behavior
Opening dropdown causes it to overflow and the panel to scroll

Expected Behavior
Opening dropdown shouldn't cause a scroll

The dropdown is a <panel-list> and looks like we have Bug 1866864 to try using the Popover API to resolve.

Whiteboard: [recomp]
Severity: -- → S4
Assignee: nobody → tgiles
Status: NEW → ASSIGNED
Priority: -- → P3

add Anna's popover patch

Ensure panel-list popovers are anchored to invoker

Make popover panel-lists open in the correct direction

Remove redundant parts of aboutLogging

Trying to figure out why the popover immediately closes when
you click and let go of the mouse button.

These changes address the issue with the new Applications setting group
where opening the dropdown would cause the group to increase in height
due to the dropdown menu elements. Users would need to scroll down
further to see the options in this menu, and then the group would shrink
in height after selecting an option from the menu.

To solve this issue, we use the popover API on the panel-list custom
element. The popover API makes the panel-list behave like the menupopup
custom element, which is the behavior we want. Now opening a select menu
in the Applications section causes the associated panel-list to appear
on top of the group, but anchored to the element that opened it.

However, to ensure that the panel-list appears anchored to the opening
element, we need to override some of the default UA styles for
[popover]. By default, a popover appears centered on the window, so we
override the margin to ensure the panel-list appears anchored to the
correct element.

Attachment #9544022 - Attachment is obsolete: true

Since the MenuController from moz-button handles the panel-list state
logic, we don't need this duplicate handler in about:logging.

Attachment #9544023 - Attachment is obsolete: true
Attachment #9542959 - Attachment description: WIP: Bug 2010864 - Attempt to fix about:logging after popover change. r=#recomp-reviewers → Bug 2010864 - Remove duplicate event handler in about:logging. r=#recomp-reviewers
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 149 Branch

Authored by https://github.com/TGiles
https://github.com/mozilla/enterprise-firefox/commit/3f109fa808858020956de3474f9350569e3f42b6
[enterprise-main] Bug 2010864 - Add support for popover in panel-list. r=desktop-theme-reviewers,mkennedy,emilio

Authored by https://github.com/TGiles
https://github.com/mozilla/enterprise-firefox/commit/518d3cc5e8b7c3baa626c1b34d3629d98c353bbf
[enterprise-main] Bug 2010864 - Remove duplicate event handler in about:logging. r=mkennedy

Blocks: 2017668
Regressions: 2018563
QA Whiteboard: [qa-ver-opt-c150/b149][qa-triage-done-c150/b149]

This reverts commit 3f109fa808858020956de3474f9350569e3f42b6.

Attachment #9549656 - Flags: approval-mozilla-beta?
Attachment #9549656 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Verified as fixed on Firefox Nightly 150.0a1 and Firefox 149.0b2 on Windows 10, Ubuntu 22, macOS 13.

Opening the dropdown does not cause scrolling.

Please note that the issue could not be reproduced, as builds prior to 2026-02-13 do not contain the new Settings UI design, and the issue is no longer reproducible on 2026-02-13 Firefox Nightly builds.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-ver-opt-c150/b149][qa-triage-done-c150/b149] → [qa-ver-done-c150/b149][qa-triage-done-c150/b149]
Duplicate of this bug: 1866864
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: