Closes mozilla-mobile#25942: hide navigation buttons from synced hist…

…ory empty state
mavduevskiy · Jul 26, 2022 · cfecd1a · cfecd1a
1 parent e33c10a
commit cfecd1a
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/app/src/main/java/org/mozilla/fenix/library/history/HistoryView.kt b/app/src/main/java/org/mozilla/fenix/library/history/HistoryView.kt
@@ -27,7 +27,7 @@ class HistoryView(
     val interactor: HistoryInteractor,
     val onZeroItemsLoaded: () -> Unit,
     val onEmptyStateChanged: (Boolean) -> Unit,
-    val isSyncedHistory: Boolean
+    val isSyncedHistory: Boolean,
 ) : LibraryPageView(container), UserInteractionHandler {
 
     val binding = ComponentHistoryBinding.inflate(
@@ -123,7 +123,7 @@ class HistoryView(
     private fun updateEmptyState(userHasHistory: Boolean) {
         binding.historyList.isInvisible = !userHasHistory
         binding.historyEmptyView.isVisible = !userHasHistory
-        binding.topSpacer.isVisible = !userHasHistory
+        binding.topSpacer.isVisible = !isSyncedHistory && !userHasHistory
 
         with(binding.recentlyClosedNavEmpty) {
             recentlyClosedNav.setOnClickListener {
@@ -140,14 +140,15 @@ class HistoryView(
                 ),
                 numRecentTabs
             )
-            recentlyClosedNav.isVisible = !userHasHistory
+            recentlyClosedNav.isVisible = !isSyncedHistory && !userHasHistory
         }
 
         with(binding.syncedHistoryNavEmpty) {
             syncedHistoryNav.setOnClickListener {
                 interactor.onSyncedHistoryClicked()
             }
-            syncedHistoryNav.isVisible = FeatureFlags.showSyncedHistory && !userHasHistory
+            syncedHistoryNav.isVisible =
+                !isSyncedHistory && FeatureFlags.showSyncedHistory && !userHasHistory
         }
 
         if (!userHasHistory) {