commit e821559f74fdbcff1b9d7434ade855a5991ec25b parent 27dd8169b11e76e1011ce02f0392c0b98c8a607a Author: Akhil Pindiprolu <apindiprolu@mozilla.com> Date: Tue, 6 Jan 2026 15:58:57 +0000 Bug 1993064 - Add kdocs/comments explaining why the Tab Manager's `TabRow` is wrapped by a `CenterAlignedTopAppBar` r=android-reviewers,jdelorenzo,007 Differential Revision: https://phabricator.services.mozilla.com/D277701 Diffstat:
| M | mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tabstray/ui/banner/TabsTrayBanner.kt | | | 4 | ++++ |
1 file changed, 4 insertions(+), 0 deletions(-)
diff --git a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tabstray/ui/banner/TabsTrayBanner.kt b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/tabstray/ui/banner/TabsTrayBanner.kt @@ -259,6 +259,10 @@ private fun TabPageBanner( val inactiveColor = MaterialTheme.colorScheme.onSurfaceVariant val selectedTabIndex = Page.pageToPosition(selectedPage) + // We wrap the TabRow in a TopAppBar to reuse Material3's built-in scroll behavior. + // CenterAlignedTopAppBar provides the scroll-to-collapse behavior via `scrollBehavior`, + // which TabRow/PrimaryTabRow does not support on its own. Without this wrapper, we'd have + // to duplicate the app bar scroll behavior implementation here. CenterAlignedTopAppBar( title = { Column(