fix: hide patch button

ReVanced · Sep 20, 2023 · 65a3d78 · 65a3d78
1 parent c4b9bb1
commit 65a3d78
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 9 deletions.
diff --git a/app/src/main/java/app/revanced/manager/ui/screen/PatchesSelectorScreen.kt b/app/src/main/java/app/revanced/manager/ui/screen/PatchesSelectorScreen.kt
@@ -101,16 +101,17 @@ fun PatchesSelectorScreen(
             )
         },
         floatingActionButton = {
-            ExtendedFloatingActionButton(
-                text = { Text(stringResource(R.string.patch)) },
-                icon = { Icon(Icons.Default.Build, null) },
-                onClick = {
-                    composableScope.launch {
-                        // TODO: only allow this if all required options have been set.
-                        onPatchClick(vm.getAndSaveSelection(), vm.getOptions())
+            if (!vm.isSelectionEmpty()) {
+                ExtendedFloatingActionButton(
+                    text = { Text(stringResource(R.string.patch)) },
+                    icon = { Icon(Icons.Default.Build, null) },
+                    onClick = {
+                        composableScope.launch {
+                            onPatchClick(vm.getAndSaveSelection(), vm.getOptions())
+                        }
                     }
-                }
-            )
+                )
+            }
         }
     ) { paddingValues ->
         Column(

diff --git a/app/src/main/java/app/revanced/manager/ui/viewmodel/PatchesSelectorViewModel.kt b/app/src/main/java/app/revanced/manager/ui/viewmodel/PatchesSelectorViewModel.kt
@@ -123,6 +123,8 @@ class PatchesSelectorViewModel(
         if (patches.contains(name)) patches.remove(name) else patches.add(name)
     }
 
+    fun isSelectionEmpty() = selectedPatches.values.all { it.isEmpty() }
+
     suspend fun getAndSaveSelection(): PatchesSelection =
         selectedPatches.also {
             withContext(Dispatchers.Default) {