Merge pull request #5046 from element-hq/feature/bma/a11y/createRoomImage
[a11y] Improve accessibility on avatar when creating a room.
This commit is contained in:
commit
566377abc8
1 changed files with 11 additions and 1 deletions
|
|
@ -30,6 +30,8 @@ import androidx.compose.ui.Alignment
|
||||||
import androidx.compose.ui.Modifier
|
import androidx.compose.ui.Modifier
|
||||||
import androidx.compose.ui.platform.LocalFocusManager
|
import androidx.compose.ui.platform.LocalFocusManager
|
||||||
import androidx.compose.ui.res.stringResource
|
import androidx.compose.ui.res.stringResource
|
||||||
|
import androidx.compose.ui.semantics.clearAndSetSemantics
|
||||||
|
import androidx.compose.ui.semantics.contentDescription
|
||||||
import androidx.compose.ui.text.input.KeyboardCapitalization
|
import androidx.compose.ui.text.input.KeyboardCapitalization
|
||||||
import androidx.compose.ui.tooling.preview.PreviewParameter
|
import androidx.compose.ui.tooling.preview.PreviewParameter
|
||||||
import androidx.compose.ui.unit.dp
|
import androidx.compose.ui.unit.dp
|
||||||
|
|
@ -213,11 +215,19 @@ private fun RoomNameWithAvatar(
|
||||||
horizontalArrangement = Arrangement.spacedBy(16.dp),
|
horizontalArrangement = Arrangement.spacedBy(16.dp),
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
) {
|
) {
|
||||||
|
val a11yAvatar = stringResource(CommonStrings.a11y_room_avatar)
|
||||||
UnsavedAvatar(
|
UnsavedAvatar(
|
||||||
avatarUri = avatarUri,
|
avatarUri = avatarUri,
|
||||||
avatarSize = AvatarSize.EditRoomDetails,
|
avatarSize = AvatarSize.EditRoomDetails,
|
||||||
avatarType = AvatarType.Room(),
|
avatarType = AvatarType.Room(),
|
||||||
modifier = Modifier.clickable(onClick = onAvatarClick),
|
modifier = Modifier
|
||||||
|
.clickable(
|
||||||
|
onClick = onAvatarClick,
|
||||||
|
onClickLabel = stringResource(CommonStrings.action_open_context_menu),
|
||||||
|
)
|
||||||
|
.clearAndSetSemantics {
|
||||||
|
contentDescription = a11yAvatar
|
||||||
|
},
|
||||||
)
|
)
|
||||||
|
|
||||||
TextField(
|
TextField(
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue