🐛 FIX(UI): Use hashCode() for song keys in Library and Home screens t…

…o prevent recomposition issues.
maxrave-dev · Jan 29, 2025 · a252b84 · a252b84
1 parent 53c84cc
commit a252b84
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/src/main/java/com/maxrave/simpmusic/ui/screen/home/HomeScreen.kt b/app/src/main/java/com/maxrave/simpmusic/ui/screen/home/HomeScreen.kt
@@ -742,7 +742,7 @@ fun ChartData(
                         state = lazyListState1,
                         flingBehavior = snapperFlingBehavior1,
                     ) {
-                        items(chart.songs, key = { it.videoId }) {
+                        items(chart.songs, key = { it.hashCode() }) {
                             ItemTrackChart(onClick = {
                                 viewModel.setQueueData(
                                     QueueData(
@@ -777,7 +777,7 @@ fun ChartData(
             state = lazyListState,
             flingBehavior = snapperFlingBehavior,
         ) {
-            items(chart.videos.items.size, key = { index -> chart.videos.items[index].videoId }) {
+            items(chart.videos.items.size, key = { index -> chart.videos.items[index].videoId + index }) {
                 val data = chart.videos.items[it]
                 ItemVideoChart(
                     onClick = {
@@ -819,7 +819,7 @@ fun ChartData(
         ) {
             items(chart.artists.itemArtists.size, key = { index ->
                 val item = chart.artists.itemArtists[index]
-                item.title + item.browseId
+                item.title + item.browseId + index
             }) {
                 val data = chart.artists.itemArtists[it]
                 ItemArtistChart(onClick = {

diff --git a/app/src/main/java/com/maxrave/simpmusic/ui/screen/library/LibraryDynamicPlaylistScreen.kt b/app/src/main/java/com/maxrave/simpmusic/ui/screen/library/LibraryDynamicPlaylistScreen.kt
@@ -81,7 +81,7 @@ fun LibraryDynamicPlaylistScreen(
                     LibraryDynamicPlaylistType.MostPlayed -> mostPlayed
                     else -> emptyList()
                 },
-                key = { it.videoId },
+                key = { it.hashCode() },
             ) { song ->
                 SongFullWidthItems(
                     songEntity = song,