Don't expose the tab tray to the browser

mozilla-mobile · Apr 20, 2020 · 1136eb4 · 1136eb4
1 parent ad1bb13
commit 1136eb4
Showing 1 changed file with 14 additions and 13 deletions.
diff --git a/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarController.kt b/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarController.kt
@@ -14,6 +14,7 @@ import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.MainScope
+import kotlinx.coroutines.delay
 import kotlinx.coroutines.launch
 import mozilla.components.browser.session.Session
 import mozilla.components.browser.session.SessionManager
@@ -316,19 +317,19 @@ class DefaultBrowserToolbarController(
     }
 
     private fun animateTabAndNavigateHome() {
-        navController.navigate(BrowserFragmentDirections.actionBrowserFragmentToTabsTrayFragment())
-//        scope.launch {
-//            browserAnimator.beginAnimateOut()
-//            // Delay for a short amount of time so the browser has time to start animating out
-//            // before we transition the fragment. This makes the animation feel smoother
-//            delay(ANIMATION_DELAY)
-//            if (!navController.popBackStack(R.id.homeFragment, false)) {
-//                navController.nav(
-//                    R.id.browserFragment,
-//                    BrowserFragmentDirections.actionGlobalHome()
-//                )
-//            }
-//        }
+        // navController.navigate(BrowserFragmentDirections.actionBrowserFragmentToTabsTrayFragment())
+        scope.launch {
+            browserAnimator.beginAnimateOut()
+            // Delay for a short amount of time so the browser has time to start animating out
+            // before we transition the fragment. This makes the animation feel smoother
+            delay(ANIMATION_DELAY)
+            if (!navController.popBackStack(R.id.homeFragment, false)) {
+                navController.nav(
+                    R.id.browserFragment,
+                    BrowserFragmentDirections.actionGlobalHome()
+                )
+            }
+        }
     }
 
     private fun reportSiteIssue(reportUrl: String, private: Boolean) {