Adjust typography in SelectedUser component (#2768)

* Adjust typography in `SelectedUser` component

* Set 2 max lines for `SelectedUser` component.

Also update the theme token references.

* Update screenshots

---------

Co-authored-by: ElementBot <benoitm+elementbot@element.io>
This commit is contained in:
Jorge Martin Espinosa
2024-04-30 11:21:39 +02:00
committed by GitHub
parent a003fffb40
commit 2559d26dc0
44 changed files with 93 additions and 90 deletions

1
changelog.d/2760.bugfix Normal file
View File

@@ -0,0 +1 @@
Adjust the typography used in the selected user component so a user's display name fits better.

View File

@@ -25,7 +25,6 @@ import androidx.compose.foundation.layout.size
import androidx.compose.foundation.layout.width
import androidx.compose.foundation.shape.CircleShape
import androidx.compose.material.ripple.rememberRipple
import androidx.compose.material3.MaterialTheme
import androidx.compose.runtime.Composable
import androidx.compose.runtime.remember
import androidx.compose.ui.Alignment
@@ -33,8 +32,10 @@ import androidx.compose.ui.Modifier
import androidx.compose.ui.draw.clip
import androidx.compose.ui.draw.clipToBounds
import androidx.compose.ui.res.stringResource
import androidx.compose.ui.text.style.TextAlign
import androidx.compose.ui.text.style.TextOverflow
import androidx.compose.ui.unit.dp
import io.element.android.compound.theme.ElementTheme
import io.element.android.compound.tokens.generated.CompoundIcons
import io.element.android.libraries.designsystem.components.avatar.Avatar
import io.element.android.libraries.designsystem.components.avatar.AvatarSize
@@ -67,13 +68,14 @@ fun SelectedUser(
modifier = Modifier.clipToBounds(),
text = matrixUser.getBestName(),
overflow = TextOverflow.Ellipsis,
maxLines = 1,
style = MaterialTheme.typography.bodyLarge,
maxLines = 2,
style = ElementTheme.typography.fontBodyMdRegular,
textAlign = TextAlign.Center,
)
}
if (canRemove) {
Surface(
color = MaterialTheme.colorScheme.primary,
color = ElementTheme.colors.textPrimary,
modifier = Modifier
.clip(CircleShape)
.size(20.dp)
@@ -87,7 +89,7 @@ fun SelectedUser(
Icon(
imageVector = CompoundIcons.Close(),
contentDescription = stringResource(id = CommonStrings.action_remove),
tint = MaterialTheme.colorScheme.onPrimary,
tint = ElementTheme.colors.iconOnSolidPrimary,
modifier = Modifier.padding(2.dp)
)
}
@@ -99,7 +101,7 @@ fun SelectedUser(
@Composable
internal fun SelectedUserPreview() = ElementPreview {
SelectedUser(
aMatrixUser(),
aMatrixUser(displayName = "John Doe"),
canRemove = true,
onUserRemoved = {},
)