diff --git a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarCFRPresenter.kt b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarCFRPresenter.kt index 321d738f1384..b6c218a3ce19 100644 --- a/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarCFRPresenter.kt +++ b/mobile/android/fenix/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarCFRPresenter.kt @@ -243,7 +243,8 @@ class BrowserToolbarCFRPresenter( } context.isTablet() && - settings.shouldShowTabletNavigationCFR -> ToolbarCFR.TABLET_NAVIGATION + settings.shouldShowTabletNavigationCFR && + settings.navigationToolbarEnabled -> ToolbarCFR.TABLET_NAVIGATION shoppingExperienceFeature.isEnabled && settings.shouldShowReviewQualityCheckCFR -> whichShoppingCFR() @@ -519,7 +520,11 @@ class BrowserToolbarCFRPresenter( ), popupVerticalOffset = CFR_TO_ANCHOR_VERTICAL_PADDING.dp, dismissButtonColor = getColor(context, R.color.fx_mobile_icon_color_oncolor), - indicatorDirection = CFRPopup.IndicatorDirection.UP, + indicatorDirection = if (settings.toolbarPosition == ToolbarPosition.TOP) { + CFRPopup.IndicatorDirection.UP + } else { + CFRPopup.IndicatorDirection.DOWN + }, ), onDismiss = { AddressToolbar.tabletNavigationCfrDismissed.record(NoExtras()) diff --git a/mobile/android/fenix/app/src/test/java/org/mozilla/fenix/components/toolbar/BrowserToolbarCFRPresenterTest.kt b/mobile/android/fenix/app/src/test/java/org/mozilla/fenix/components/toolbar/BrowserToolbarCFRPresenterTest.kt index 5334a4e11c90..61def52fec5a 100644 --- a/mobile/android/fenix/app/src/test/java/org/mozilla/fenix/components/toolbar/BrowserToolbarCFRPresenterTest.kt +++ b/mobile/android/fenix/app/src/test/java/org/mozilla/fenix/components/toolbar/BrowserToolbarCFRPresenterTest.kt @@ -489,6 +489,7 @@ class BrowserToolbarCFRPresenterTest { every { shouldShowEraseActionCFR } returns false every { shouldShowReviewQualityCheckCFR } returns false every { shouldShowTabletNavigationCFR } returns true + every { navigationToolbarEnabled } returns true }, )