Closed Bug 2057169 Opened 1 month ago Closed 21 days ago

Top and bottom menu items have their corners cut off by the menupopup's larger border radius

Categories

(Toolkit :: Themes, defect)

defect

Tracking

()

VERIFIED FIXED
155 Branch
Tracking Status
firefox155 --- verified

People

(Reporter: dao, Assigned: dwhisman)

References

(Blocks 1 open bug)

Details

(Whiteboard: [recomp-nova][acorntractors])

Attachments

(3 files)

No description provided.
See Also: → 2057771
Duplicate of this bug: 2058257
Duplicate of this bug: 2057170

See bug 2057170 comment 1 from Jules:

This menupopup uses the panel border radius token which with Nova on is now a roundier value. Some of our latest token changes have affected it consequently.

The line is here. We should instead not share styles and make a custom property or token for menupopup border radius and go back to the smaller radius to guarantee it never changes based on what we do with our other panels.

WIP patch has some changes for revamping the token usage so --popup- would be for base menupopup, panel and --panel- would be for [type=arrow]. I think that's likely something we want to do going forward but it could use some more thinking through regarding making this distinction more explicit between the different use cases and expanding it to more properties

For now we should consider just modifying the popup.css rules to revert the Nova radius getting increased

Duplicate of this bug: 2058471
Assignee: nobody → dwhisman
Status: NEW → ASSIGNED
Attachment #9622254 - Attachment description: WIP: Bug 2057169 - Prevent menu items from being cut off by menupopup border radius r=#desktop-theme-reviewers → Bug 2057169 - Prevent menu items from being cut off by menupopup border radius r=#desktop-theme-reviewers
Duplicate of this bug: 2061937
Duplicate of this bug: 2061957
Duplicate of this bug: 2060984
Pushed by dwhisman@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/fc9537321c2e https://hg.mozilla.org/integration/autoland/rev/ccd290239c10 Prevent menu items from being cut off by menupopup border radius r=desktop-theme-reviewers,emilio
Status: ASSIGNED → RESOLVED
Closed: 21 days ago
Resolution: --- → FIXED
Target Milestone: --- → 155 Branch

Reproduced the initial issue and the ones from the duplicate bugs using Nightly 155.0a1 (Build ID: 20260723204011) on Ubuntu 24.04.

Verified as fixed using the latest Nightly 155.0a1 on Ubuntu 24.04 - the top and bottom of the menus are not cut anymore.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-ver-done-c155/b154]
QA Contact: sbadau
See Also: → 2063827
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: