1974977 - allow badging non-native menu items with "New"

Status: VERIFIED FIXED

(Toolkit :: Themes, enhancement)

Description

[Ed Lee :Mardak]

like bug 1970651 for native macos menus and split the css change from bug 1971178

Comment 1

[Ed Lee :Mardak]

Attachment: Bug 1974977 - allow badging non-native menu items with "New" r=emilio!

Split menu styling from D253104.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/f6054c20b1e5
https://hg.mozilla.org/integration/autoland/rev/54b25c7a1866
allow badging non-native menu items with "New" r=emilio,desktop-theme-reviewers

Comment 3

[[e-mail]]

https://hg.mozilla.org/mozilla-central/rev/54b25c7a1866

Comment 4

[Rares Doghi, Desktop QA]

This issue is verified as fixed on Windows and Linux in our latest Nightly 142.0a1 (2025-07-02) , Mac Os < 14 still has this issue, I cannot see the new badge on a MacOs 11.7

Comment 5

[Ed Lee :Mardak]

Attachment: Bug 1974977 - allow badging non-native menu items with "New"

Split menu styling from D253104.

Original Revision: https://phabricator.services.mozilla.com/D255636

Comment 6

[Phabricator Automation]

firefox-beta Uplift Approval Request

• User impact if declined: windows users don't see "New" on summarize page menu item available for macos
• Code covered by automated testing: no
• Fix verified in Nightly: yes
• Needs manual QE test: yes
• Steps to reproduce for manual QE testing: 1. enable feature with browser.ml.chat.page 2. open page context menu
• Risk associated with taking this patch: low
• Explanation of risk level: css addition for badge attribute that's currently only activated from page summarize experiment
• String changes made/needed: no
• Is Android affected?: no

Comment 7

[Pulsebot]

https://github.com/mozilla-firefox/firefox/commit/e3bd27c73c05
https://hg.mozilla.org/releases/mozilla-beta/rev/9b528f30dd9a

Comment 8

[Rares Doghi, Desktop QA]

This issue is verified as fixed in our latest Beta 141.0b6.