From fa2d9825f2feff05285bd321cc8843b65a65f6e7 Mon Sep 17 00:00:00 2001
From: MM20 <15646950+MM2-0@users.noreply.github.com>
Date: Wed, 21 Sep 2022 17:39:57 +0200
Subject: [PATCH] Hide frequently used rows slider when frequently used switch
 is disabled

---
 .../ui/launcher/modals/EditFavoritesSheet.kt  | 68 ++++++++++---------
 1 file changed, 37 insertions(+), 31 deletions(-)

diff --git a/ui/src/main/java/de/mm20/launcher2/ui/launcher/modals/EditFavoritesSheet.kt b/ui/src/main/java/de/mm20/launcher2/ui/launcher/modals/EditFavoritesSheet.kt
index 7ecee11a..9fe8275f 100644
--- a/ui/src/main/java/de/mm20/launcher2/ui/launcher/modals/EditFavoritesSheet.kt
+++ b/ui/src/main/java/de/mm20/launcher2/ui/launcher/modals/EditFavoritesSheet.kt
@@ -283,40 +283,46 @@ fun ReorderFavoritesGrid(viewModel: EditFavoritesSheetVM) {
                                             }
                                         )
                                     }
-                                    Column(
-                                        modifier = Modifier
-                                            .fillMaxWidth()
-                                            .padding(top = 8.dp)
-                                            .padding(horizontal = 16.dp),
-                                        horizontalAlignment = Alignment.Start
-                                    ) {
-                                        Text(
-                                            modifier = Modifier.fillMaxWidth(),
-                                            text = "Number of rows",
-                                            style = MaterialTheme.typography.labelMedium
-                                        )
-                                        Row(
-                                            modifier = Modifier.fillMaxWidth(),
-                                            verticalAlignment = Alignment.CenterVertically
+                                    AnimatedVisibility(enableFrequentlyUsed == true) {
+                                        Column(
+                                            modifier = Modifier
+                                                .fillMaxWidth()
+                                                .padding(top = 8.dp)
+                                                .padding(horizontal = 16.dp),
+                                            horizontalAlignment = Alignment.Start
                                         ) {
-                                            Slider(
-                                                modifier = Modifier.weight(1f).padding(end = 16.dp),
-                                                value = frequentlyUsedRows.toFloat(),
-                                                colors = SliderDefaults.colors(
-                                                    inactiveTrackColor = MaterialTheme.colorScheme.onSurfaceVariant
-                                                ),
-                                                onValueChange = {
-                                                    viewModel.setFrequentlyUsedRows(it.roundToInt())
-                                                },
-                                                steps = 2,
-                                                valueRange = 1f..4f
-                                            )
                                             Text(
-                                                text = frequentlyUsedRows.toString(),
-                                                modifier = Modifier.width(52.dp).padding(4.dp),
-                                                style = MaterialTheme.typography.labelMedium,
-                                                textAlign = TextAlign.Center
+                                                modifier = Modifier.fillMaxWidth(),
+                                                text = "Number of rows",
+                                                style = MaterialTheme.typography.labelMedium
                                             )
+                                            Row(
+                                                modifier = Modifier.fillMaxWidth(),
+                                                verticalAlignment = Alignment.CenterVertically
+                                            ) {
+                                                Slider(
+                                                    modifier = Modifier
+                                                        .weight(1f)
+                                                        .padding(end = 16.dp),
+                                                    value = frequentlyUsedRows.toFloat(),
+                                                    colors = SliderDefaults.colors(
+                                                        inactiveTrackColor = MaterialTheme.colorScheme.onSurfaceVariant
+                                                    ),
+                                                    onValueChange = {
+                                                        viewModel.setFrequentlyUsedRows(it.roundToInt())
+                                                    },
+                                                    steps = 2,
+                                                    valueRange = 1f..4f
+                                                )
+                                                Text(
+                                                    text = frequentlyUsedRows.toString(),
+                                                    modifier = Modifier
+                                                        .width(52.dp)
+                                                        .padding(4.dp),
+                                                    style = MaterialTheme.typography.labelMedium,
+                                                    textAlign = TextAlign.Center
+                                                )
+                                            }
                                         }
                                     }
                                 }