[Spike] Investigate how to remove the remaining flash in top sites
Categories
(Firefox for Android :: Top Sites, defect)
Tracking
()
People
(Reporter: jonalmeida, Unassigned)
Details
From github: https://github.com/mozilla-mobile/fenix/issues/15180.
Title
How to remove the short flash in all top sites when removing one.
Description
From the discussion in https://github.com/mozilla-mobile/fenix/pull/15126, we should investigate into generating stable IDs that can be used across the two data sources (pinned and frecent), and then use that for diff-ing the layout for
DiffCallback.areItemsTheSame.Deliverables
Description of deliverables, including:
Findings - Are the stable IDs enough to solve the problem?
Next Steps - If yes, fix it! Otherwise, create a new issue to look into reducing the complexity of the top sites nested adapters by writing a custom ViewPager or LayoutManager.┆Issue is synchronized with this Jira Task
Change performed by the Move to Bugzilla add-on.
|
Reporter | |
Comment 1•3 years ago
|
||
With the birth of Jetpack Compose, we should just re-write our top sites to use Compose instead of trying to reason with an outdated API for nested layouts.
|
||
Comment 2•3 years ago
|
||
The severity field is not set for this bug.
:cpeterson, could you have a look please?
For more information, please visit auto_nag documentation.
|
|
Reporter | |
Comment 3•3 years ago
|
||
Closing as per comment 1.
Description
•