Sulkta-OSS/element-x-ada
3
0
Fork 0
Code Issues 1 Pull requests 10 Projects Releases Packages Wiki Activity Actions

Remove default param for AvatarCluster.avatarType

Browse source
This commit is contained in:
Benoit Marty 2025-06-23 21:27:40 +02:00
parent f3a885e535
commit b532f6e60b
2 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/components/avatar/AvatarCluster.kt
View file

@ -33,8 +33,8 @@ private const val MAX_AVATAR_COUNT = 4
@Composable @Composable
internal fun AvatarCluster( internal fun AvatarCluster(
avatars: ImmutableList<AvatarData>, avatars: ImmutableList<AvatarData>,
avatarType: AvatarType,
modifier: Modifier = Modifier, modifier: Modifier = Modifier,
avatarType: AvatarType = AvatarType.User,
hideAvatarImages: Boolean = false, hideAvatarImages: Boolean = false,
contentDescription: String? = null, contentDescription: String? = null,
) { ) {
@ -119,6 +119,7 @@ internal fun AvatarClusterPreview() = ElementThemedPreview {
for (ngOfAvatars in 1..5) { for (ngOfAvatars in 1..5) {
AvatarCluster( AvatarCluster(
avatars = List(ngOfAvatars) { anAvatarData(it) }.toPersistentList(), avatars = List(ngOfAvatars) { anAvatarData(it) }.toPersistentList(),
avatarType = AvatarType.User,
) )
} }
} }

2
libraries/designsystem/src/main/kotlin/io/element/android/libraries/designsystem/components/avatar/RoomAvatar.kt
View file

@ -49,6 +49,8 @@ internal fun RoomAvatar(
else -> { else -> {
AvatarCluster( AvatarCluster(
avatars = avatarType.heroes, avatars = avatarType.heroes,
// Note: even for a room avatar, we use UserAvatarType here to display the avatar of heroes
avatarType = AvatarType.User,
modifier = modifier, modifier = modifier,
hideAvatarImages = hideAvatarImage, hideAvatarImages = hideAvatarImage,
contentDescription = contentDescription contentDescription = contentDescription