Merge pull request #7136 from element-hq/fix/a11y-invite-people

Announce checked/unchecked state for user rows in invite screen
This commit is contained in:
Benoit Marty
2026-08-07 14:00:01 +02:00
committed by GitHub
@@ -8,12 +8,12 @@
package io.element.android.libraries.matrix.ui.components package io.element.android.libraries.matrix.ui.components
import androidx.compose.foundation.clickable
import androidx.compose.foundation.layout.Column import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.Row import androidx.compose.foundation.layout.Row
import androidx.compose.foundation.layout.Spacer import androidx.compose.foundation.layout.Spacer
import androidx.compose.foundation.layout.fillMaxWidth import androidx.compose.foundation.layout.fillMaxWidth
import androidx.compose.foundation.layout.width import androidx.compose.foundation.layout.width
import androidx.compose.foundation.selection.toggleable
import androidx.compose.runtime.Composable import androidx.compose.runtime.Composable
import androidx.compose.runtime.Immutable import androidx.compose.runtime.Immutable
import androidx.compose.ui.Alignment import androidx.compose.ui.Alignment
@@ -40,8 +40,8 @@ fun CheckableUserRow(
Row( Row(
modifier = modifier modifier = modifier
.fillMaxWidth() .fillMaxWidth()
.clickable(role = Role.Checkbox, enabled = enabled) { .toggleable(value = checked, role = Role.Checkbox, enabled = enabled) {
onCheckedChange(!checked) onCheckedChange(it)
}, },
verticalAlignment = Alignment.CenterVertically, verticalAlignment = Alignment.CenterVertically,
) { ) {
@@ -66,7 +66,7 @@ fun CheckableUserRow(
} }
} }
Checkbox( Checkbox(
onCheckedChange = onCheckedChange, onCheckedChange = null,
checked = checked, checked = checked,
enabled = enabled, enabled = enabled,
) )