[Jetcaster] Add on click label to podcast follow button

riyol · Jul 16, 2021 · 6a6aa57 · 6a6aa57
1 parent 89e4437
commit 6a6aa57
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/Jetcaster/app/src/main/java/com/example/jetcaster/util/Buttons.kt b/Jetcaster/app/src/main/java/com/example/jetcaster/util/Buttons.kt
@@ -33,6 +33,8 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.draw.shadow
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.res.stringResource
+import androidx.compose.ui.semantics.onClick
+import androidx.compose.ui.semantics.semantics
 import androidx.compose.ui.unit.dp
 import com.example.jetcaster.R
 
@@ -42,9 +44,12 @@ fun ToggleFollowPodcastIconButton(
     onClick: () -> Unit,
     modifier: Modifier = Modifier
 ) {
+    val clickLabel = stringResource(if (isFollowed) R.string.cd_unfollow else R.string.cd_follow)
     IconButton(
         onClick = onClick,
-        modifier = modifier
+        modifier = modifier.semantics {
+            onClick(label = clickLabel, action = null)
+        }
     ) {
         Icon(
             // TODO: think about animating these icons
@@ -53,8 +58,8 @@ fun ToggleFollowPodcastIconButton(
                 else -> Icons.Default.Add
             },
             contentDescription = when {
-                isFollowed -> stringResource(R.string.cd_unfollow)
-                else -> stringResource(R.string.cd_follow)
+                isFollowed -> stringResource(R.string.cd_following)
+                else -> stringResource(R.string.cd_not_following)
             },
             tint = animateColorAsState(
                 when {

diff --git a/Jetcaster/app/src/main/res/values/strings.xml b/Jetcaster/app/src/main/res/values/strings.xml
@@ -47,5 +47,7 @@
     <string name="cd_play">Play</string>
     <string name="cd_unfollow">Unfollow</string>
     <string name="cd_follow">Follow</string>
+    <string name="cd_following">Following</string>
+    <string name="cd_not_following">Not following</string>
 
 </resources>