Audio item design lift

alashow · Jul 23, 2021 · eb8016a · eb8016a
1 parent 71fea2a
commit eb8016a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/common-ui-components/src/main/java/tm/alashow/ui/components/Image.kt b/common-ui-components/src/main/java/tm/alashow/ui/components/Image.kt
@@ -66,7 +66,7 @@ fun ImageWithPlaceholder(
 @Composable
 fun CoverImage(
     painter: LoadPainter<Any>,
-    size: Dp = 50.dp,
+    size: Dp = 48.dp,
     backgroundColor: Color = MaterialTheme.colors.surface,
     shape: Shape = MaterialTheme.shapes.small,
     icon: VectorPainter = rememberVectorPainter(Icons.Default.MusicNote),

diff --git a/core-ui-media/src/main/java/tm/alashow/datmusic/ui/audios/AudioRow.kt b/core-ui-media/src/main/java/tm/alashow/datmusic/ui/audios/AudioRow.kt
@@ -27,6 +27,7 @@ import androidx.compose.ui.Modifier
 import androidx.compose.ui.composed
 import androidx.compose.ui.graphics.graphicsLayer
 import androidx.compose.ui.text.style.TextOverflow
+import androidx.compose.ui.unit.dp
 import com.google.accompanist.coil.rememberCoilPainter
 import com.google.accompanist.placeholder.PlaceholderHighlight
 import com.google.accompanist.placeholder.material.placeholder
@@ -51,6 +52,7 @@ fun AudioRow(
 
     Row(
         horizontalArrangement = Arrangement.SpaceBetween,
+        verticalAlignment = Alignment.CenterVertically,
         modifier = modifier
             .clickable {
                 if (!isPlaceholder)
@@ -101,6 +103,7 @@ fun AudioRowItem(
     )
     Row(
         horizontalArrangement = Arrangement.spacedBy(AppTheme.specs.padding),
+        verticalAlignment = Alignment.CenterVertically,
         modifier = modifier
     ) {
         val image = rememberCoilPainter(audio.coverUrlSmall ?: audio.coverUrl, fadeIn = true)
@@ -112,7 +115,7 @@ fun AudioRowItem(
             )
         }
 
-        Column(verticalArrangement = Arrangement.spacedBy(AppTheme.specs.paddingSmall)) {
+        Column(verticalArrangement = Arrangement.spacedBy(if (isPlaceholder) AppTheme.specs.paddingTiny else 0.dp)) {
             Text(
                 audio.title,
                 style = MaterialTheme.typography.body1,