Closed Bug 1700503 Opened 6 months ago Closed 6 months ago

Add enterprise policy support menu to app menu

Categories

(Firefox :: Menus, task, P1)

task

Tracking

()

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: emalysz, Assigned: emalysz)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-hamburger-menu] )

Attachments

(1 file)

In Bug 1527766, we added this button to the help menu in the menubar, but it is not correctly added to our app menu's help menu.

We actually log an error here: https://searchfox.org/mozilla-central/source/browser/components/customizableui/content/panelUI.js#715-718 because we don't have an appmenu-data-l10n-id for that menuitem.

UX/Product specified it should go in between Report deceptive site… and About Firefox (Windows & Linux) (only available on Linux/Windows)

Report Deceptive Site...
{Enterprise support menu}
About Firefox (Windows & Linux)

Since it's not going to have the same position as it does in the menubar (where it's last), we need to assign it a specific position behind appMenu_menu_HelpPopup_reportPhishingtoolmenu.
While we're here, we should also update browser_policy_support_menu.js to check the app menu.

Assignee: nobody → emalysz
Status: NEW → ASSIGNED

Marking as P1 because it's fixing an l10n error.

Priority: -- → P1
Pushed by emalysz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4b264b981f12
insert enterprise policy support menu item to app menu r=mkaply
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.