Closed Bug 1904445 Opened 5 months ago Closed 2 months ago

[Menu Redesign] Launch the new menu redesign from browser without relying on the toolbar redesign

Categories

(Fenix :: Toolbar, task, P1)

All
Android
task

Tracking

(firefox131 fixed)

RESOLVED FIXED
131 Branch
Tracking Status
firefox131 --- fixed

People

(Reporter: sanchaai, Assigned: gl)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

The task here is to ensure we can launch new menu redesign from the browser as long as the menu redesign secret setting is turned on. This means we want to be able to launch the new menu redesign from the browser using the old toolbar as well as the new toolbar whenever the menu redesign secret setting is turned on.

Severity: -- → N/A
Priority: -- → P1
Duplicate of this bug: 1909881
Assignee: nobody → gl
Status: NEW → ASSIGNED

Originally, the sessionId is only provided when there's a custom tab. Otherwise, it is always null if it is a normal browser session.
Instead of calling this sessionId where it might imply that this could be a normal browser session, let's rename this to customTabId to make it very clear that this is either a custom tab id or null if it is a normal browser session.

Attachment #9419152 - Attachment description: WIP: Bug 1904445 - Part 1: Refactor reader mode action and feature into its own init function in BrowserFragment → Bug 1904445 - Part 1: Refactor reader mode action and feature into its own init function in BrowserFragment
Attachment #9419153 - Attachment description: WIP: Bug 1904445 - Part 2: Rename sessionId to customTabId in MenuPresenter and ToolbarIntegration implementations → Bug 1904445 - Part 2: Rename sessionId to customTabId in MenuPresenter and ToolbarIntegration implementations
Attachment #9419154 - Attachment description: WIP: Bug 1904445 - Part 3: Add Menu browser action in the browser and custom tab toolbar integration → Bug 1904445 - Part 3: Add Menu browser action in the browser and custom tab toolbar integration
Attachment #9419155 - Attachment description: WIP: Bug 1904445 - Part 4: Add telemetry for clicking on the browser menu action → Bug 1904445 - Part 4: Add telemetry for clicking on the browser menu action
Blocks: 1909881
No longer duplicate of this bug: 1909881
Keywords: leave-open
Attachment #9419153 - Attachment description: Bug 1904445 - Part 2: Rename sessionId to customTabId in MenuPresenter and ToolbarIntegration implementations → Bug 1904445 - Part 2: Rename sessionId to customTabId in MenuPresenter, BrowserToolbarCFRPresenter and ToolbarIntegration implementations
Pushed by gluong@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f7699969c86c Part 1: Refactor reader mode action and feature into its own init function in BrowserFragment r=android-reviewers,petru https://hg.mozilla.org/integration/autoland/rev/1af5b1f3335d Part 2: Rename sessionId to customTabId in MenuPresenter, BrowserToolbarCFRPresenter and ToolbarIntegration implementations r=android-reviewers,petru https://hg.mozilla.org/integration/autoland/rev/cf5c914fd778 Part 3: Add Menu browser action in the browser and custom tab toolbar integration r=android-reviewers,petru
Attachment #9419155 - Attachment description: Bug 1904445 - Part 4: Add telemetry for clicking on the browser menu action → Bug 1904445 - Part 4: Record telemetry for opening the home and browser menu from the toolbar
Keywords: leave-open
Pushed by gluong@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c6478c5d93c1 Part 4: Record telemetry for opening the home and browser menu from the toolbar r=android-reviewers,petru
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 131 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: