Use LazyColumn in SearchScreen

app/src/main/kotlin/it/vfsfitvnm/vimusic/ui/screens/SearchScreen.kt

@@ -11,6 +11,7 @@ import androidx.compose.foundation.clickable
 import androidx.compose.foundation.interaction.MutableInteractionSource
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
+import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.Row
 import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.fillMaxSize
@@ -18,13 +19,13 @@ import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.height
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.size
-import androidx.compose.foundation.rememberScrollState
+import androidx.compose.foundation.lazy.LazyColumn
+import androidx.compose.foundation.lazy.items
 import androidx.compose.foundation.shape.CircleShape
 import androidx.compose.foundation.text.BasicText
 import androidx.compose.foundation.text.BasicTextField
 import androidx.compose.foundation.text.KeyboardActions
 import androidx.compose.foundation.text.KeyboardOptions
-import androidx.compose.foundation.verticalScroll
 import androidx.compose.material.ripple.rememberRipple
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
@@ -52,6 +53,7 @@ import androidx.core.net.toUri
 import it.vfsfitvnm.route.RouteHandler
 import it.vfsfitvnm.vimusic.Database
 import it.vfsfitvnm.vimusic.R
+import it.vfsfitvnm.vimusic.models.SearchQuery
 import it.vfsfitvnm.vimusic.query
 import it.vfsfitvnm.vimusic.ui.components.TopAppBar
 import it.vfsfitvnm.vimusic.ui.components.themed.LoadingOrError
@@ -72,6 +74,25 @@ fun SearchScreen(
     onSearch: (String) -> Unit,
     onUri: (Uri) -> Unit,
 ) {
+    val albumRoute = rememberAlbumRoute()
+    val artistRoute = rememberArtistRoute()
+
+    RouteHandler(listenToGlobalEmitter = true) {
+        albumRoute { browseId ->
+            AlbumScreen(
+                browseId = browseId ?: error("browseId cannot be null")
+            )
+        }
+
+        artistRoute { browseId ->
+            ArtistScreen(
+                browseId = browseId ?: error("browseId cannot be null")
+            )
+        }
+
+        host {
+            val (colorPalette, typography) = LocalAppearance.current
+
             var textFieldValue by rememberSaveable(initialTextInput, stateSaver = TextFieldValue.Saver) {
                 mutableStateOf(
                     TextFieldValue(
@@ -104,24 +125,6 @@ fun SearchScreen(
                 }
             }.collectAsState(initial = null, context = Dispatchers.IO)
 
-    val albumRoute = rememberAlbumRoute()
-    val artistRoute = rememberArtistRoute()
-
-    RouteHandler(listenToGlobalEmitter = true) {
-        albumRoute { browseId ->
-            AlbumScreen(
-                browseId = browseId ?: error("browseId cannot be null")
-            )
-        }
-
-        artistRoute { browseId ->
-            ArtistScreen(
-                browseId = browseId ?: error("browseId cannot be null")
-            )
-        }
-
-        host {
-            val (colorPalette, typography) = LocalAppearance.current
 
             val isOpenableUrl = remember(textFieldValue.text) {
                 listOf(
@@ -264,12 +267,13 @@ fun SearchScreen(
                     }
                 }
 
-                Column(
-                    modifier = Modifier
-                        .verticalScroll(rememberScrollState())
-                        .padding(bottom = Dimensions.collapsedPlayer)
+                LazyColumn(
+                    contentPadding = PaddingValues(bottom = Dimensions.collapsedPlayer)
                 ) {
-                    history?.forEach { searchQuery ->
+                    items(
+                        items = history ?: emptyList(),
+                        key = SearchQuery::id
+                    ) { searchQuery ->
                         Row(
                             verticalAlignment = Alignment.CenterVertically,
                             modifier = Modifier
@@ -332,7 +336,7 @@ fun SearchScreen(
                     }
 
                     searchSuggestionsResult?.getOrNull()?.let { suggestions ->
-                        suggestions.forEach { suggestion ->
+                        items(items = suggestions) { suggestion ->
                             Row(
                                 verticalAlignment = Alignment.CenterVertically,
                                 modifier = Modifier
@@ -378,10 +382,12 @@ fun SearchScreen(
                             }
                         }
                     } ?: searchSuggestionsResult?.exceptionOrNull()?.let { throwable ->
+                        item {
                             LoadingOrError(errorMessage = throwable.javaClass.canonicalName) {}
                         }
                     }
                 }
             }
         }
+    }
 }