Antoine_Jaury_KSK/Chom-Bev-e-Brezhoneg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: improve accessibility for previous/next buttons in player

Browse Source
This commit is contained in:
Antoine Jaury
2026-10-12 14:29:59 +02:00
parent 8dc1e8e71e
commit 8339ee128b
3 changed files with 30 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
feature/records/src/commonMain/kotlin/bzh/ajaury/chombev/player/ui/components/NextButton.kt
+14 -1
View File
@@ -8,10 +8,13 @@ import androidx.compose.material3.IconButton
import androidx.compose.material3.MaterialTheme
import androidx.compose.runtime.Composable
import androidx.compose.ui.Modifier
import androidx.compose.ui.semantics.onClick
import androidx.compose.ui.semantics.semantics
import androidx.compose.ui.tooling.preview.Preview
import androidx.compose.ui.unit.dp
import bzh.ajaury.chombev.resources.generated.resources.Res
import bzh.ajaury.chombev.resources.generated.resources.player_next_sentence
import bzh.ajaury.chombev.resources.generated.resources.player_next_sentence_action_label
import org.jetbrains.compose.resources.stringResource
@Composable
@@ -20,8 +23,18 @@ fun NextButton(
modifier: Modifier = Modifier,
onNextSentenceClicked: () -> Unit = {},
) {
val actionLabel = stringResource(Res.string.player_next_sentence_action_label)
IconButton(
modifier = modifier.size(60.dp),
modifier = modifier
.size(60.dp)
.semantics {
if (canGoToNextSentence) {
onClick(label = actionLabel) {
onNextSentenceClicked()
true
}
}
},
enabled = canGoToNextSentence,
onClick = onNextSentenceClicked,
) {
feature/records/src/commonMain/kotlin/bzh/ajaury/chombev/player/ui/components/PreviousButton.kt
+14 -1
View File
@@ -8,10 +8,13 @@ import androidx.compose.material3.IconButton
import androidx.compose.material3.MaterialTheme
import androidx.compose.runtime.Composable
import androidx.compose.ui.Modifier
import androidx.compose.ui.semantics.onClick
import androidx.compose.ui.semantics.semantics
import androidx.compose.ui.tooling.preview.Preview
import androidx.compose.ui.unit.dp
import bzh.ajaury.chombev.resources.generated.resources.Res
import bzh.ajaury.chombev.resources.generated.resources.player_previous_sentence
import bzh.ajaury.chombev.resources.generated.resources.player_previous_sentence_action_label
import org.jetbrains.compose.resources.stringResource
@Composable
@@ -20,8 +23,18 @@ fun PreviousButton(
modifier: Modifier = Modifier,
onPreviousSentenceClicked: () -> Unit = {},
) {
val actionLabel = stringResource(Res.string.player_previous_sentence_action_label)
IconButton(
modifier = modifier.size(60.dp),
modifier = modifier
.size(60.dp)
.semantics {
if (canGoToPreviousSentence) {
onClick(label = actionLabel) {
onPreviousSentenceClicked()
true
}
}
},
enabled = canGoToPreviousSentence,
onClick = onPreviousSentenceClicked,
) {