Open Bug 1814225 (tabs-tray-interactor-refactor) Opened 2 years ago Updated 1 year ago

[Meta] Refactor Tabs Tray's interactors

Categories

(Fenix :: Tabs, enhancement, P2)

All
Android
enhancement

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: gl, Assigned: 007)

References

(Depends on 4 open bugs)

Details

(Keywords: meta)

From github: https://github.com/mozilla-mobile/fenix/issues/25722.

Per https://github.com/mozilla-mobile/fenix/pull/25636#discussion_r903805714 and https://github.com/mozilla-mobile/fenix/pull/25636#discussion_r903843844, the Tabs Tray's interactors should be consolidated into one top-level interactor so there is a clearer call stack for handling user interactions. This way there is one entry point for everything Tabs Tray-related functionality and only one dependency passed down into front-end code consumers.

This effort should be split into the following issues:

Phase 1 (minor refactoring noise and quick wins

Phase 2 (substantive refactors)

**Phase 3 (follow-up)

  • [ ] Create an interactor/controller for the FAB
  • [ ] Refactor the sync tabs portion of the tabs tray FAB to live in the FAB interactor/controller flow
  • [ ] Decouple TabsTrayInteractor from TabsTray.Delegate after #21318

┆Issue is synchronized with this Jira Task

Change performed by the Move to Bugzilla add-on.

Assignee: nobody → nbond
Status: NEW → ASSIGNED
Summary: Refactor Tabs Tray's interactors → [Meta] Refactor Tabs Tray's interactors
Alias: tabs-tray-interactor-refactor
Depends on: 1814224, 1814223, 1814206
Depends on: 1814241, 1814243
Type: defect → enhancement
Depends on: 1814506
Severity: -- → S3
Priority: -- → P2
Severity: S3 → N/A
Depends on: 1822887
Depends on: 1822878
Depends on: 1824029
Depends on: 1824030
Depends on: 1824199
Depends on: 1826453
Depends on: 1835102
Depends on: 1835123
You need to log in before you can comment on or make changes to this bug.