Add an "Add to group" item to the new browser menu
Categories
(Firefox for Android :: Menu, task)
Tracking
()
People
(Reporter: petru, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxdroid][android-core])
The legacy browser menu (MenuDialogFragment) had an "Add to group" item in its "More" submenu (MoreSettingsSubmenu).
It was shown when the pref_key_show_tab_groups_in_menu secret setting (Settings.showTabGroupsInMenu) was enabled, and opened the tab group flow for the selected tab through MenuAction.Navigate.OpenTabGroupFlow with TabGroupFlowEntryPoint.AddTabToGroup.
The customizable menu (MenuFragment) never got an equivalent item.
Bug 2090842 would remove the legacy menu along with its "Add to group" item and the browser_menu_add_to_tab_group string, so the secret setting now has no effect and the tab group flow cannot be reached from the menu.
Expected: the browser menu offers "Add to group" for the selected tab, opening TabGroupFlowFragment with TabGroupFlowEntryPoint.AddTabToGroup.
Implementation notes:
- Add a menu item provider for it, registered in
MenuItemProvidersFactoryand placed inBrowserMenuConfiguration. It can navigate with the existingaction_global_tabGroupFlowFragmentglobal action. - Reuse the
showTabGroupsInMenugate. It defaults to off because enabling it pushed Print out of the visible menu area, which the Print UI tests rely on (seeDefaultTabManagementFeatureHelper); check whether that still applies to the new menu layout. MenuAction.Navigate.OpenTabGroupFlowand its telemetry branch inMenuTelemetryMiddleware(recordingTabGroupFlowEntryPoint.telemetryName) are still in place and can be reused.
Updated•2 days ago
|
| Reporter | ||
Updated•2 days ago
|
Description
•