Closed Bug 1958392 Opened 1 year ago Closed 1 year ago

The context menu for a downloaded file in the Downloads panel has too much inline padding

Categories

(Firefox :: Menus, defect)

defect

Tracking

()

VERIFIED FIXED
139 Branch
Tracking Status
firefox-esr128 --- unaffected
firefox137 --- unaffected
firefox138 --- unaffected
firefox139 --- verified

People

(Reporter: itiel_yn8, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files, 1 obsolete file)

Attached image screenshot.png

This happens because the downloadsContextMenu menupopup lives under the <panel>, causing this to set the extra margin.
I think the ultimate solution would be to move downloadsContextMenu out of the panel, while making sure this isn't relevant anymore since then.

Otherwise it'd be possible to override the vars for this panel, but it feels like a gross hack...

Tentatively marking bug 1954977 as the regressor, I have no idea if this came up recently but I figured it was too much of a coincidence that I found out about this just now.
I have no means of using mozregression on this machine so I can't be sure, but if it turns out this isn't the regressor feel free to remove the tag/keyword.

Keywords: regression
Regressed by: 1954977
Summary: menupopup that opens when right clicking a downloaded file in the Downloads panel have too much inline padding → The context menu for a downloaded file in the Downloads panel has too much inline padding

Set release status flags based on info from the regressing bug 1954977

:emilio, since you are the author of the regressor, bug 1954977, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Attached image screenshot (obsolete) —

Bug appears in ordinary menu popups as well as contextual menus.

Sorry this is Bug 1958396.

Attachment #9477118 - Attachment is obsolete: true

This is a more consistent with the other --panel-* variables.

Assignee: nobody → emilio
Status: NEW → ASSIGNED
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2ab1e94eb4a3 Reset menuitem variables at the menupopup/panel boundaries. r=desktop-theme-reviewers,dao
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 139 Branch
QA Whiteboard: [qa-triage-done-c140/b139][qa-verif-needed-c140/b139]
Flags: qe-verify+
QA Whiteboard: [qa-triage-done-c140/b139][qa-verif-needed-c140/b139] → [qa-triage-done-c140/b139][qa-ver-needed-c140/b139]
QA Contact: bmaris

Reproduced the issue from comment 0 using an old Nightly build from 2025-04-02. Verified that using latest Nightly 140.0a1 and Firefox Beta 139.0b1 on Windows 11 (and also checked Ubuntu 22.04 and MacOS 13) the extra padding is no longer seen in the download context menu.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-triage-done-c140/b139][qa-ver-needed-c140/b139] → [qa-triage-done-c140/b139][qa-ver-done-c140/b139]
Flags: qe-verify+
Blocks: 1964959
Duplicate of this bug: 1964959
No longer duplicate of this bug: 1964959
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: