From f33cb691cf882397224817798f8008d7f9edd6a1 Mon Sep 17 00:00:00 2001
From: rushiiMachine <33725716+rushiiMachine@users.noreply.github.com>
Date: Mon, 4 Mar 2024 09:56:53 -0800
Subject: [PATCH] feat(HomeScreen): minor layout adjustments

---
 .../manager/ui/screens/home/HomeScreen.kt     | 81 ++++++++-----------
 .../screens/home/components/InstallButton.kt  |  8 ++
 .../home/components/InstalledItemCard.kt      |  2 +-
 app/src/main/res/values/strings.xml           |  2 +-
 4 files changed, 45 insertions(+), 48 deletions(-)

diff --git a/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/HomeScreen.kt b/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/HomeScreen.kt
index 3ab9d3f..958c29d 100644
--- a/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/HomeScreen.kt
+++ b/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/HomeScreen.kt
@@ -15,7 +15,6 @@ import androidx.compose.runtime.*
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.draw.alpha
-import androidx.compose.ui.draw.shadow
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp
@@ -131,6 +130,7 @@ fun PresentInstallsContent(
 
         item(key = "ADD_INSTALL_BUTTON") {
             InstallButton(
+                secondaryInstall = true,
                 onClick = onClickInstall,
                 modifier = Modifier
                     .fillMaxWidth()
@@ -141,7 +141,7 @@ fun PresentInstallsContent(
         item(key = "SUPPORTED_VERSION") {
             AnimatedVersionDisplay(
                 version = model.supportedVersion,
-                modifier = Modifier.padding(bottom = 22.dp),
+                modifier = Modifier.padding(bottom = 30.dp),
             )
         }
 
@@ -166,6 +166,21 @@ fun PresentInstallsContent(
                 )
             }
         }
+
+        item(key = "END_INDICATOR") {
+            Box(
+                contentAlignment = Alignment.Center,
+                modifier = Modifier
+                    .alpha(.6f)
+                    .padding(top = 12.dp)
+                    .fillMaxWidth(),
+            ) {
+                Text(
+                    text = "...",
+                    style = MaterialTheme.typography.labelSmall,
+                )
+            }
+        }
     }
 }
 
@@ -185,6 +200,7 @@ fun NoInstallsContent(
         ProjectHeader()
 
         InstallButton(
+            secondaryInstall = false,
             onClick = onClickInstall,
             modifier = Modifier
                 .fillMaxWidth()
@@ -193,54 +209,27 @@ fun NoInstallsContent(
 
         AnimatedVersionDisplay(
             version = supportedVersion,
-            modifier = Modifier.padding(bottom = 22.dp),
+            modifier = Modifier.padding(bottom = 30.dp),
         )
 
-        Box(
-            modifier = Modifier.fillMaxSize(),
+        Column(
+            verticalArrangement = Arrangement.spacedBy(12.dp, Alignment.CenterVertically),
+            horizontalAlignment = Alignment.CenterHorizontally,
+            modifier = Modifier
+                .alpha(.7f)
+                .padding(bottom = 90.dp)
+                .fillMaxSize()
         ) {
-            Column(
-                verticalArrangement = Arrangement.spacedBy(6.dp),
-                modifier = Modifier
-                    .alpha(.1f)
-                    .fillMaxSize()
-            ) {
-                for (i in 0..<3) key(i) {
-                    Surface(
-                        content = {},
-                        shape = MaterialTheme.shapes.medium,
-                        tonalElevation = 2.dp,
-                        modifier = Modifier
-                            .fillMaxWidth()
-                            .height(195.dp)
-                            .shadow(
-                                clip = false,
-                                elevation = 2.dp,
-                                shape = MaterialTheme.shapes.medium,
-                            ),
-                    )
-                }
-            }
+            Text(
+                text = """ /ᐠﹷ ‸ ﹷ ᐟ\ﾉ""",
+                style = MaterialTheme.typography.labelLarge
+                    .copy(fontSize = 38.sp),
+            )
 
-            Column(
-                verticalArrangement = Arrangement.spacedBy(14.dp),
-                horizontalAlignment = Alignment.CenterHorizontally,
-                modifier = Modifier
-                    .alpha(.8f)
-                    .align(Alignment.Center)
-                    .padding(bottom = 80.dp),
-            ) {
-                Text(
-                    text = """ /ᐠﹷ ‸ ﹷ ᐟ\ﾉ""",
-                    style = MaterialTheme.typography.labelLarge
-                        .copy(fontSize = 38.sp),
-                )
-
-                Text(
-                    text = stringResource(R.string.installs_no_installs),
-                    style = MaterialTheme.typography.labelMedium,
-                )
-            }
+            Text(
+                text = stringResource(R.string.installs_no_installs),
+                style = MaterialTheme.typography.labelLarge,
+            )
         }
     }
 }
diff --git a/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/components/InstallButton.kt b/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/components/InstallButton.kt
index eb8ed1c..8d48f21 100644
--- a/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/components/InstallButton.kt
+++ b/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/components/InstallButton.kt
@@ -14,11 +14,19 @@ import com.aliucord.manager.R
 @Composable
 fun InstallButton(
     enabled: Boolean = true,
+    secondaryInstall: Boolean,
     onClick: () -> Unit,
     modifier: Modifier = Modifier,
 ) {
     FilledTonalIconButton(
         shape = MaterialTheme.shapes.medium,
+        colors = IconButtonDefaults.filledTonalIconButtonColors(
+            containerColor = if (secondaryInstall) {
+                MaterialTheme.colorScheme.secondaryContainer
+            } else {
+                MaterialTheme.colorScheme.primary
+            },
+        ),
         enabled = enabled,
         onClick = onClick,
         modifier = modifier,
diff --git a/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/components/InstalledItemCard.kt b/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/components/InstalledItemCard.kt
index 170c41a..5f59aa2 100644
--- a/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/components/InstalledItemCard.kt
+++ b/app/src/main/kotlin/com/aliucord/manager/ui/screens/home/components/InstalledItemCard.kt
@@ -43,7 +43,7 @@ fun InstalledItemCard(
             )
     ) {
         Column(
-            verticalArrangement = Arrangement.spacedBy(20.dp),
+            verticalArrangement = Arrangement.spacedBy(24.dp),
             modifier = Modifier.padding(20.dp),
         ) {
             Row(
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 3ed961e..a33a044 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -34,7 +34,7 @@
   <string name="permissions_grant_title">Grant Permissions</string>
   <string name="permissions_grant_body">In order for Aliucord Manager to function, file permissions are required. Since shared data is stored in ~/Aliucord, permissions are required in order to access it.</string>
 
-  <string name="installs_no_installs">No installations founds!</string>
+  <string name="installs_no_installs">No installations found!</string>
   <string name="launch_aliucord_fail">Failed to launch Aliucord</string>
 
   <string name="settings_theme">Theme</string>