Change variable name to remove nameShadowed warning (#95)

* Change variable name to remove `nameShadowed` warning

* Fix HomeScreenTest.kt by adding `@ExperimentalAnimationApi` to setUp

Author: wiryadev

app/src/androidTest/java/com/guru/composecookbook/ui/home/HomeScreenTest.kt

@@ -1,5 +1,6 @@
 package com.guru.composecookbook.ui.home
 
+import androidx.compose.animation.ExperimentalAnimationApi
 import androidx.compose.foundation.ExperimentalFoundationApi
 import androidx.compose.material.ExperimentalMaterialApi
 import androidx.compose.runtime.mutableStateOf
@@ -22,6 +23,7 @@ class HomeScreenTest {
     @get: Rule
     val composeAndroidTestRule = createAndroidComposeRule<MainActivity>()
 
+    @ExperimentalAnimationApi
     @ExperimentalFoundationApi
     @ExperimentalMaterialApi
     @Before

app/src/main/java/com/guru/composecookbook/ui/home/advancelists/SwipeableLists.kt

@@ -36,15 +36,18 @@ import kotlin.math.roundToInt
 @ExperimentalMaterialApi
 @Composable
 fun SwipeableLists() {
-    val albums by mutableStateOf(AlbumsDataProvider.albums)
+    val albums by remember {
+        mutableStateOf(AlbumsDataProvider.albums)
+    }
     LazyColumn {
         itemsIndexed(
             items = albums,
             itemContent = { index, album ->
                 SwipeableListItem(index, album) { index ->
 
                 }
-            })
+            }
+        )
     }
 }
 

components/charts/src/main/java/com/guru/composecookbook/charts/Charts.kt

@@ -148,20 +148,20 @@ fun LineChart(
 
     Canvas(modifier = modifier.padding(8.dp)) {
         val path = Path()
-        val xbounds = Pair(0f, xTarget)
-        val ybounds = getBounds(yValues)
-        val scaleX = size.width / (xbounds.second - xbounds.first)
-        val scaleY = size.height / (ybounds.second - ybounds.first)
-        val yMove = ybounds.first * scaleY
+        val xBounds = Pair(0f, xTarget)
+        val yBounds = getBounds(yValues)
+        val scaleX = size.width / (xBounds.second - xBounds.first)
+        val scaleY = size.height / (yBounds.second - yBounds.first)
+        val yMove = yBounds.first * scaleY
 
         (0..min(yValues.size - 1, x.value.toInt())).forEach { value ->
-            val x = value * scaleX
-            val y = size.height - (yValues[value] * scaleY) + yMove
+            val xPoint = value * scaleX
+            val yPoint = size.height - (yValues[value] * scaleY) + yMove
             if (value == 0) {
-                path.moveTo(0f, y)
+                path.moveTo(0f, yPoint)
                 return@forEach
             }
-            path.lineTo(x, y)
+            path.lineTo(xPoint, yPoint)
         }
 
         drawPath(
@@ -194,16 +194,21 @@ fun BarCharts(
     }
 
     Canvas(modifier = modifier.padding(horizontal = 8.dp)) {
-        val xbounds = Pair(0f, xTarget)
-        val ybounds = getBounds(yValues)
-        val scaleX = size.width / (xbounds.second - xbounds.first)
-        val scaleY = size.height / (ybounds.second - ybounds.first)
-        val yMove = ybounds.first * scaleY
+        val xBounds = Pair(0f, xTarget)
+        val yBounds = getBounds(yValues)
+        val scaleX = size.width / (xBounds.second - xBounds.first)
+        val scaleY = size.height / (yBounds.second - yBounds.first)
+        val yMove = yBounds.first * scaleY
 
         (0..min(yValues.size - 1, x.value.toInt())).forEach { value ->
-            val x = value * scaleX
-            val y = size.height - (yValues[value] * scaleY) + yMove
-            drawBar(topLeft = Offset(x, y), width = barWidth, height = size.height - y, barColors)
+            val xOffset = value * scaleX
+            val yOffset = size.height - (yValues[value] * scaleY) + yMove
+            drawBar(
+                topLeft = Offset(xOffset, yOffset),
+                width = barWidth,
+                height = size.height - yOffset,
+                barColors
+            )
         }
     }
 }

demos/datingapp/src/main/java/com/guru/composecookbook/datingapp/components/home/DatingHomeScreen.kt

@@ -65,9 +65,9 @@ fun DatingHomeScreen() {
                             start = 16.dp,
                             end = 16.dp
                         ),
-                    { _, album ->
+                    onSwiped = { _, swipedAlbum ->
                         if (persons.isNotEmpty()) {
-                            persons.remove(album)
+                            persons.remove(swipedAlbum)
                             if (persons.isEmpty()) {
                                 listEmpty.value = true
                             }

demos/spotify/src/main/java/com/guru/composecookbook/spotify/ui/home/SpotifyActivity.kt

@@ -87,8 +87,8 @@ fun SpotifyBottomNavigation(spotifyNavItemState: MutableState<SpotifyNavType>) {
 
 @Composable
 fun SpotifyBodyContent(spotifyNavType: SpotifyNavType) {
-    Crossfade(targetState = spotifyNavType) { spotifyNavType ->
-        when (spotifyNavType) {
+    Crossfade(targetState = spotifyNavType) { navType ->
+        when (navType) {
             SpotifyNavType.HOME -> SpotifyHome()
             SpotifyNavType.SEARCH -> SpotifySearchScreen()
             SpotifyNavType.LIBRARY -> SpotifyPlayList()