Просмотр исходного кода

Add vertical padding to items in lists

vfsfitvnm 3 лет назад
Родитель
Сommit
73cc0554c2

+ 5 - 5
app/src/main/kotlin/it/vfsfitvnm/vimusic/ui/screens/SearchResultScreen.kt

@@ -343,7 +343,7 @@ fun SmallItem(
                     interactionSource = remember { MutableInteractionSource() },
                     interactionSource = remember { MutableInteractionSource() },
                     onClick = onClick
                     onClick = onClick
                 )
                 )
-                .padding(vertical = 5.dp, horizontal = 16.dp)
+                .padding(vertical = Dimensions.itemsVerticalPadding, horizontal = 16.dp)
         )
         )
         is YouTube.Item.Song -> SmallSongItem(
         is YouTube.Item.Song -> SmallSongItem(
             song = item,
             song = item,
@@ -361,7 +361,7 @@ fun SmallItem(
                     interactionSource = remember { MutableInteractionSource() },
                     interactionSource = remember { MutableInteractionSource() },
                     onClick = onClick
                     onClick = onClick
                 )
                 )
-                .padding(vertical = 5.dp, horizontal = 16.dp)
+                .padding(vertical = Dimensions.itemsVerticalPadding, horizontal = 16.dp)
         )
         )
         is YouTube.Item.Video -> SmallVideoItem(
         is YouTube.Item.Video -> SmallVideoItem(
             video = item,
             video = item,
@@ -379,7 +379,7 @@ fun SmallItem(
                     interactionSource = remember { MutableInteractionSource() },
                     interactionSource = remember { MutableInteractionSource() },
                     onClick = onClick
                     onClick = onClick
                 )
                 )
-                .padding(vertical = 5.dp, horizontal = 16.dp)
+                .padding(vertical = Dimensions.itemsVerticalPadding, horizontal = 16.dp)
         )
         )
     }
     }
 }
 }
@@ -584,7 +584,7 @@ private fun LoadingOrError(
                     modifier = Modifier
                     modifier = Modifier
                         .alpha(1f - index * 0.125f)
                         .alpha(1f - index * 0.125f)
                         .fillMaxWidth()
                         .fillMaxWidth()
-                        .padding(vertical = 5.dp, horizontal = 16.dp)
+                        .padding(vertical = Dimensions.itemsVerticalPadding, horizontal = 16.dp)
                 )
                 )
             } else {
             } else {
                 SmallSongItemShimmer(
                 SmallSongItemShimmer(
@@ -592,7 +592,7 @@ private fun LoadingOrError(
                     modifier = Modifier
                     modifier = Modifier
                         .alpha(1f - index * 0.125f)
                         .alpha(1f - index * 0.125f)
                         .fillMaxWidth()
                         .fillMaxWidth()
-                        .padding(vertical = 5.dp, horizontal = 16.dp)
+                        .padding(vertical = Dimensions.itemsVerticalPadding, horizontal = 16.dp)
                 )
                 )
             }
             }
         }
         }

+ 2 - 0
app/src/main/kotlin/it/vfsfitvnm/vimusic/ui/styling/Dimensions.kt

@@ -8,6 +8,8 @@ import androidx.compose.ui.unit.dp
 
 
 @Suppress("ClassName")
 @Suppress("ClassName")
 object Dimensions {
 object Dimensions {
+    val itemsVerticalPadding = 8.dp
+
     object thumbnails {
     object thumbnails {
         val album = 128.dp
         val album = 128.dp
         val artist = 192.dp
         val artist = 192.dp

+ 1 - 1
app/src/main/kotlin/it/vfsfitvnm/vimusic/ui/views/SongItem.kt

@@ -170,7 +170,7 @@ fun SongItem(
                 onClick = onClick
                 onClick = onClick
             )
             )
             .fillMaxWidth()
             .fillMaxWidth()
-            .padding(vertical = 5.dp)
+            .padding(vertical = Dimensions.itemsVerticalPadding)
             .background(backgroundColor ?: colorPalette.background)
             .background(backgroundColor ?: colorPalette.background)
             .padding(start = 16.dp, end = if (trailingContent == null) 16.dp else 8.dp)
             .padding(start = 16.dp, end = if (trailingContent == null) 16.dp else 8.dp)
     ) {
     ) {