For mozilla-mobile#6980: Update Private browsing card.

Added test to description. Added link to settings. Updated unit tests.
mcarare · Feb 12, 2020 · 669658a · 669658a
1 parent 8fd9b65
commit 669658a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt
@@ -550,8 +550,7 @@ class HomeFragment : Fragment() {
     }
 
     private fun openSettingsScreen() {
-        hideOnboardingIfNeeded()
-        val directions = HomeFragmentDirections.actionHomeFragmentToSettingsFragment()
+        val directions = HomeFragmentDirections.actionHomeFragmentToPrivateBrowsingFragment()
         nav(R.id.homeFragment, directions)
     }
 

diff --git a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/SessionControlView.kt b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/SessionControlView.kt
@@ -110,8 +110,8 @@ private fun onboardingAdapterItems(onboardingState: OnboardingState): List<Adapt
             it.getString(R.string.onboarding_feature_section_header, appName)
         },
         AdapterItem.OnboardingWhatsNew,
-        AdapterItem.OnboardingThemePicker,
         AdapterItem.OnboardingTrackingProtection,
+        AdapterItem.OnboardingThemePicker,
         AdapterItem.OnboardingPrivateBrowsing,
         AdapterItem.OnboardingToolbarPositionPicker,
         AdapterItem.OnboardingPrivacyNotice,

diff --git a/app/src/main/res/navigation/nav_graph.xml b/app/src/main/res/navigation/nav_graph.xml
@@ -92,6 +92,9 @@
         <action
             android:id="@+id/action_homeFragment_to_shareFragment"
             app:destination="@id/shareFragment" />
+        <action
+            android:id="@+id/action_homeFragment_to_privateBrowsingFragment"
+            app:destination="@id/privateBrowsingFragment" />
     </fragment>
 
     <fragment