Closes mozilla-mobile#804: Adds disable private browsing content desc…

…ription
sblatz · Mar 5, 2019 · 7ef76c0 · 7ef76c0
1 parent fca5ed3
commit 7ef76c0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 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
@@ -157,6 +157,13 @@ class HomeFragment : Fragment() {
 
         view.toolbar_wrapper.isPrivateModeEnabled = (activity as HomeActivity).browsingModeManager.isPrivate
 
+        if ((activity as HomeActivity).browsingModeManager.isPrivate) {
+            privateBrowsingButton.contentDescription =
+                    getString(R.string.content_description_disable_private_browsing_button)
+        } else {
+                privateBrowsingButton.contentDescription = getString(R.string.content_description_private_browsing_button)
+        }
+
         privateBrowsingButton.setOnClickListener {
             val browsingModeManager = (activity as HomeActivity).browsingModeManager
             browsingModeManager.mode = when (browsingModeManager.mode) {

diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -10,6 +10,8 @@
     <string name="content_description_menu">More options</string>
     <!-- Content description (not visible, for screen readers etc.): "Private Browsing" menu button. -->
     <string name="content_description_private_browsing_button">Enable private browsing</string>
+    <!-- Content description (not visible, for screen readers etc.): "Private Browsing" menu button. -->
+    <string name="content_description_disable_private_browsing_button">Disable private browsing</string>
     <!-- Explanation for how sessions work -->
     <string name="sessions_intro_description">Sessions help you return to the sites you visit often and those you\'ve just discovered. Start browsing and they\'ll begin to appear here. </string>
     <!-- Placeholder text shown in the search bar before a user enters text -->