diff --git a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt index a52870f21..9d46a6f6e 100644 --- a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt @@ -190,7 +190,7 @@ class HomeFragment : Fragment() { closeTab = ::closeTab, closeAllTabs = ::closeAllTabs, getListOfTabs = ::getListOfTabs, - hideOnboarding = ::hideOnboarding, + hideOnboarding = ::hideOnboardingAndOpenSearch, invokePendingDeleteJobs = ::invokePendingDeleteJobs, registerCollectionStorageObserver = ::registerCollectionStorageObserver, scrollToTheTop = ::scrollToTheTop, @@ -285,10 +285,7 @@ class HomeFragment : Fragment() { view.add_tab_button.setOnClickListener { invokePendingDeleteJobs() hideOnboardingIfNeeded() - val directions = HomeFragmentDirections.actionHomeFragmentToSearchFragment( - sessionId = null - ) - nav(R.id.homeFragment, directions) + navigateToSearch() } PrivateBrowsingButtonView( @@ -482,15 +479,25 @@ class HomeFragment : Fragment() { } private fun hideOnboardingIfNeeded() { - if (!onboarding.userHasBeenOnboarded()) hideOnboarding() + if (!onboarding.userHasBeenOnboarded()) { + onboarding.finish() + homeFragmentStore.dispatch( + HomeFragmentAction.ModeChange( + mode = currentMode.getCurrentMode(), + tabs = getListOfSessions().toTabs())) + } } - private fun hideOnboarding() { - onboarding.finish() - homeFragmentStore.dispatch( - HomeFragmentAction.ModeChange( - mode = currentMode.getCurrentMode(), - tabs = getListOfSessions().toTabs())) + private fun hideOnboardingAndOpenSearch() { + hideOnboardingIfNeeded() + navigateToSearch() + } + + private fun navigateToSearch() { + val directions = HomeFragmentDirections.actionHomeFragmentToSearchFragment( + sessionId = null + ) + nav(R.id.homeFragment, directions) } private fun createHomeMenu(context: Context): HomeMenu { diff --git a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/SessionControlInteractor.kt b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/SessionControlInteractor.kt index a89ef75db..3995864af 100644 --- a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/SessionControlInteractor.kt +++ b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/SessionControlInteractor.kt @@ -83,7 +83,7 @@ interface CollectionInteractor { */ interface OnboardingInteractor { /** - * Hides the onboarding. Called when a user clicks on the "Start Browsing" button. + * Hides the onboarding and navigates to Search. Called when a user clicks on the "Start Browsing" button. */ fun onStartBrowsingClicked() }