From ca20996b62c893fbe47b4b4a7ce083cc00ad1079 Mon Sep 17 00:00:00 2001
From: Ax333l <main@axelen.xyz>
Date: Fri, 19 May 2023 21:21:37 +0200
Subject: [PATCH] refactor(ui): move PatchItem to the only file where it is
 used

---
 .../manager/compose/ui/component/PatchItem.kt | 47 -------------------
 .../ui/screen/PatchesSelectorScreen.kt        | 42 ++++++++++++++++-
 2 files changed, 41 insertions(+), 48 deletions(-)
 delete mode 100644 app/src/main/java/app/revanced/manager/compose/ui/component/PatchItem.kt

diff --git a/app/src/main/java/app/revanced/manager/compose/ui/component/PatchItem.kt b/app/src/main/java/app/revanced/manager/compose/ui/component/PatchItem.kt
deleted file mode 100644
index a2a45194..00000000
--- a/app/src/main/java/app/revanced/manager/compose/ui/component/PatchItem.kt
+++ /dev/null
@@ -1,47 +0,0 @@
-package app.revanced.manager.compose.ui.component
-
-import androidx.compose.foundation.clickable
-import androidx.compose.material.icons.Icons
-import androidx.compose.material.icons.outlined.Settings
-import androidx.compose.material3.*
-import androidx.compose.runtime.Composable
-import androidx.compose.ui.Modifier
-import androidx.compose.ui.draw.alpha
-import app.revanced.manager.compose.patcher.patch.PatchInfo
-
-@Composable
-fun PatchItem(
-    patch: PatchInfo,
-    onOptionsDialog: () -> Unit,
-    selected: Boolean,
-    onToggle: () -> Unit,
-    supported: Boolean
-) {
-    ListItem(
-        modifier = Modifier
-            .let { if (!supported) it.alpha(0.5f) else it }
-            .clickable(enabled = supported, onClick = onToggle),
-        leadingContent = {
-            Checkbox(
-                checked = selected,
-                onCheckedChange = {
-                    onToggle()
-                },
-                enabled = supported
-            )
-        },
-        headlineContent = {
-            Text(patch.name)
-        },
-        supportingContent = {
-            Text(patch.description ?: "")
-        },
-        trailingContent = {
-            if (patch.options?.isNotEmpty() == true) {
-                IconButton(onClick = onOptionsDialog, enabled = supported) {
-                    Icon(Icons.Outlined.Settings, null)
-                }
-            }
-        }
-    )
-}
\ No newline at end of file
diff --git a/app/src/main/java/app/revanced/manager/compose/ui/screen/PatchesSelectorScreen.kt b/app/src/main/java/app/revanced/manager/compose/ui/screen/PatchesSelectorScreen.kt
index 2cc0ba29..35d2aa09 100644
--- a/app/src/main/java/app/revanced/manager/compose/ui/screen/PatchesSelectorScreen.kt
+++ b/app/src/main/java/app/revanced/manager/compose/ui/screen/PatchesSelectorScreen.kt
@@ -1,6 +1,7 @@
 package app.revanced.manager.compose.ui.screen
 
 import androidx.compose.foundation.ExperimentalFoundationApi
+import androidx.compose.foundation.clickable
 import androidx.compose.foundation.layout.*
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.items
@@ -10,18 +11,20 @@ import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.filled.Build
 import androidx.compose.material.icons.outlined.HelpOutline
 import androidx.compose.material.icons.outlined.Search
+import androidx.compose.material.icons.outlined.Settings
 import androidx.compose.material3.*
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.getValue
 import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.ui.Modifier
+import androidx.compose.ui.draw.alpha
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.unit.dp
 import androidx.lifecycle.compose.collectAsStateWithLifecycle
 import app.revanced.manager.compose.R
+import app.revanced.manager.compose.patcher.patch.PatchInfo
 import app.revanced.manager.compose.ui.component.AppTopBar
 import app.revanced.manager.compose.ui.component.GroupHeader
-import app.revanced.manager.compose.ui.component.PatchItem
 import app.revanced.manager.compose.ui.viewmodel.PatchesSelectorViewModel
 import kotlinx.coroutines.launch
 
@@ -134,6 +137,43 @@ fun PatchesSelectorScreen(
     }
 }
 
+@Composable
+fun PatchItem(
+    patch: PatchInfo,
+    onOptionsDialog: () -> Unit,
+    selected: Boolean,
+    onToggle: () -> Unit,
+    supported: Boolean
+) {
+    ListItem(
+        modifier = Modifier
+            .let { if (!supported) it.alpha(0.5f) else it }
+            .clickable(enabled = supported, onClick = onToggle),
+        leadingContent = {
+            Checkbox(
+                checked = selected,
+                onCheckedChange = {
+                    onToggle()
+                },
+                enabled = supported
+            )
+        },
+        headlineContent = {
+            Text(patch.name)
+        },
+        supportingContent = {
+            Text(patch.description ?: "")
+        },
+        trailingContent = {
+            if (patch.options?.isNotEmpty() == true) {
+                IconButton(onClick = onOptionsDialog, enabled = supported) {
+                    Icon(Icons.Outlined.Settings, null)
+                }
+            }
+        }
+    )
+}
+
 @Composable
 fun UnsupportedDialog(
     onDismissRequest: () -> Unit