From 03a5cb2fb4253b0092c8c25f02a272e6e6283447 Mon Sep 17 00:00:00 2001
From: Emily Kager <ekager@mozilla.com>
Date: Mon, 24 Jun 2019 13:15:38 -0700
Subject: [PATCH] For #3620 - Adds automatic theme onboarding content
 description (#3629)

---
 .../viewholders/onboarding/OnboardingThemePickerViewHolder.kt | 4 ++++
 app/src/main/res/layout/onboarding_theme_picker.xml           | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/onboarding/OnboardingThemePickerViewHolder.kt b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/onboarding/OnboardingThemePickerViewHolder.kt
index 2493b0204..3f2b69bf2 100644
--- a/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/onboarding/OnboardingThemePickerViewHolder.kt
+++ b/app/src/main/java/org/mozilla/fenix/home/sessioncontrol/viewholders/onboarding/OnboardingThemePickerViewHolder.kt
@@ -52,6 +52,10 @@ class OnboardingThemePickerViewHolder(private val view: View) : RecyclerView.Vie
             radioLightTheme.performClick()
         }
 
+        val automaticTitle = view.context.getString(R.string.onboarding_theme_automatic_title)
+        val automaticSummary = view.context.getString(R.string.onboarding_theme_automatic_summary)
+        view.clickable_region_automatic.contentDescription = "$automaticTitle $automaticSummary"
+
         view.clickable_region_automatic.setOnClickListener {
             radioFollowDeviceTheme.performClick()
         }
diff --git a/app/src/main/res/layout/onboarding_theme_picker.xml b/app/src/main/res/layout/onboarding_theme_picker.xml
index cbed61406..b066c837a 100644
--- a/app/src/main/res/layout/onboarding_theme_picker.xml
+++ b/app/src/main/res/layout/onboarding_theme_picker.xml
@@ -60,7 +60,6 @@
         app:layout_constraintTop_toBottomOf="@+id/description_text"
         app:onboardingKey="@string/pref_key_light_theme" />
 
-
     <ImageButton
         android:id="@+id/theme_light_image"
         android:layout_width="wrap_content"
@@ -144,6 +143,7 @@
         android:layout_marginStart="16dp"
         android:layout_marginTop="16dp"
         android:layout_marginBottom="16dp"
+        android:contentDescription="@string/onboarding_theme_automatic_title"
         android:paddingStart="0dp"
         android:paddingEnd="8dp"
         app:layout_constraintBottom_toBottomOf="parent"