From 0095041745b7cf31ee78e7509810bdefaa785668 Mon Sep 17 00:00:00 2001 From: Don Turner Date: Fri, 22 Mar 2024 11:59:11 +0000 Subject: [PATCH] Change to toggleable Surface Change-Id: I2de9aa0da57adf298b3b0219854d1c1db63fd1fe --- .../nowinandroid/feature/foryou/ForYouScreen.kt | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/feature/foryou/src/main/kotlin/com/google/samples/apps/nowinandroid/feature/foryou/ForYouScreen.kt b/feature/foryou/src/main/kotlin/com/google/samples/apps/nowinandroid/feature/foryou/ForYouScreen.kt index e206612c3..fdae1896e 100644 --- a/feature/foryou/src/main/kotlin/com/google/samples/apps/nowinandroid/feature/foryou/ForYouScreen.kt +++ b/feature/foryou/src/main/kotlin/com/google/samples/apps/nowinandroid/feature/foryou/ForYouScreen.kt @@ -389,11 +389,6 @@ private fun SingleTopicButton( ) = trace("SingleTopicButton") { val actionLabel = "Follow or unfollow" - val toggleTopicFollowed = { onClick(topicId, !isSelected) } - val accessibleToggleTopicFollowed : () -> Boolean = { - toggleTopicFollowed() - true - } Surface( modifier = Modifier @@ -406,13 +401,18 @@ private fun SingleTopicButton( "Not following" } customActions = listOf( - CustomAccessibilityAction(actionLabel, accessibleToggleTopicFollowed) + CustomAccessibilityAction(actionLabel) { + onClick(topicId, !isSelected) + true + } ) }, shape = RoundedCornerShape(corner = CornerSize(8.dp)), color = MaterialTheme.colorScheme.surface, - selected = isSelected, - onClick = toggleTopicFollowed, + checked = isSelected, + onCheckedChange = { + onClick(topicId, it) + }, ) { Row( verticalAlignment = Alignment.CenterVertically,