Browse Source

Improve MatrixUserRow paddings

test/jme/compound-poc
Florian Renaud 2 years ago
parent
commit
d2e5e4fc45
  1. 6
      features/createroom/impl/src/main/kotlin/io/element/android/features/createroom/impl/root/CreateRoomRootView.kt
  2. 5
      libraries/matrixui/src/main/kotlin/io/element/android/libraries/matrix/ui/components/MatrixUserRow.kt

6
features/createroom/impl/src/main/kotlin/io/element/android/features/createroom/impl/root/CreateRoomRootView.kt

@ -22,7 +22,7 @@ import androidx.compose.foundation.layout.Arrangement @@ -22,7 +22,7 @@ import androidx.compose.foundation.layout.Arrangement
import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.Row
import androidx.compose.foundation.layout.fillMaxWidth
import androidx.compose.foundation.layout.heightIn
import androidx.compose.foundation.layout.height
import androidx.compose.foundation.layout.padding
import androidx.compose.foundation.lazy.LazyColumn
import androidx.compose.foundation.lazy.items
@ -220,7 +220,7 @@ fun CreateRoomSearchResultItem( @@ -220,7 +220,7 @@ fun CreateRoomSearchResultItem(
onClick: () -> Unit = {},
) {
MatrixUserRow(
modifier = modifier.heightIn(min = 56.dp),
modifier = modifier,
matrixUser = matrixUser,
avatarSize = AvatarSize.Custom(36.dp),
onClick = onClick,
@ -237,7 +237,7 @@ fun CreateRoomActionButton( @@ -237,7 +237,7 @@ fun CreateRoomActionButton(
Row(
modifier = modifier
.fillMaxWidth()
.heightIn(min = 56.dp)
.height(56.dp)
.clickable { onClick() }
.padding(horizontal = 16.dp),
horizontalArrangement = Arrangement.spacedBy(16.dp),

5
libraries/matrixui/src/main/kotlin/io/element/android/libraries/matrix/ui/components/MatrixUserRow.kt

@ -41,7 +41,6 @@ import io.element.android.libraries.designsystem.theme.components.Text @@ -41,7 +41,6 @@ import io.element.android.libraries.designsystem.theme.components.Text
import io.element.android.libraries.matrix.ui.model.MatrixUser
import io.element.android.libraries.matrix.ui.model.getBestName
// FIXME Row are not the same height if there is a name or not.
@Composable
fun MatrixUserRow(
matrixUser: MatrixUser,
@ -53,7 +52,7 @@ fun MatrixUserRow( @@ -53,7 +52,7 @@ fun MatrixUserRow(
modifier = modifier
.clickable(onClick = onClick)
.fillMaxWidth()
.padding(horizontal = 16.dp)
.padding(start = 16.dp, top = 8.dp, end = 16.dp, bottom = 8.dp)
.height(IntrinsicSize.Min),
verticalAlignment = Alignment.CenterVertically
) {
@ -62,7 +61,7 @@ fun MatrixUserRow( @@ -62,7 +61,7 @@ fun MatrixUserRow(
)
Column(
modifier = Modifier
.padding(start = 12.dp, end = 4.dp, top = 12.dp, bottom = 12.dp),
.padding(start = 12.dp),
) {
// Name
Text(

Loading…
Cancel
Save