fix: finish activity when pressing back from home screen

Author: sds100

app/src/main/java/io/github/sds100/keymapper/home/HomeFragment.kt

@@ -48,6 +48,7 @@ class HomeFragment : Fragment() {
                             onAboutClick = {
                                 findNavController().navigate(NavAppDirections.actionGlobalAboutFragment())
                             },
+                            finishActivity = requireActivity()::finish,
                         )
                     }
                 }

app/src/main/java/io/github/sds100/keymapper/home/HomeScreen.kt

@@ -122,6 +122,7 @@ import androidx.navigation.compose.composable
 import androidx.navigation.compose.currentBackStackEntryAsState
 import androidx.navigation.compose.rememberNavController
 import io.github.sds100.keymapper.R
+import io.github.sds100.keymapper.backup.ImportExportState
 import io.github.sds100.keymapper.backup.RestoreType
 import io.github.sds100.keymapper.compose.KeyMapperTheme
 import io.github.sds100.keymapper.compose.LocalCustomColorsPalette
@@ -143,6 +144,7 @@ fun HomeScreen(
     viewModel: HomeViewModel,
     onSettingsClick: () -> Unit,
     onAboutClick: () -> Unit,
+    finishActivity: () -> Unit,
 ) {
     val homeState by viewModel.state.collectAsStateWithLifecycle()
 
@@ -279,7 +281,11 @@ fun HomeScreen(
                 onImportClick = { importFileLauncher.launch(FileUtils.MIME_TYPE_ALL) },
                 onTogglePausedClick = viewModel::onTogglePausedClick,
                 onFixWarningClick = viewModel::onFixWarningClick,
-                onBackClick = viewModel::onBackClick,
+                onBackClick = {
+                    if (!viewModel.onBackClick()) {
+                        finishActivity()
+                    }
+                },
                 onSelectAllClick = viewModel::onSelectAllClick,
             )
         },

app/src/main/java/io/github/sds100/keymapper/home/HomeViewModel.kt

@@ -14,6 +14,7 @@ import androidx.lifecycle.viewModelScope
 import io.github.sds100.keymapper.Constants
 import io.github.sds100.keymapper.R
 import io.github.sds100.keymapper.backup.BackupRestoreMappingsUseCase
+import io.github.sds100.keymapper.backup.ImportExportState
 import io.github.sds100.keymapper.backup.RestoreType
 import io.github.sds100.keymapper.floating.FloatingLayoutsState
 import io.github.sds100.keymapper.floating.ListFloatingLayoutsUseCase
@@ -490,8 +491,13 @@ class HomeViewModel(
         _importExportState.value = ImportExportState.Idle
     }
 
-    fun onBackClick() {
-        multiSelectProvider.stopSelecting()
+    fun onBackClick(): Boolean {
+        if (multiSelectProvider.state.value is SelectionState.Selecting) {
+            multiSelectProvider.stopSelecting()
+            return true
+        } else {
+            return false
+        }
     }
 
     @Suppress("UNCHECKED_CAST")
@@ -521,17 +527,6 @@ class HomeViewModel(
     }
 }
 
-sealed class ImportExportState {
-    data object Idle : ImportExportState()
-    data object Exporting : ImportExportState()
-    data class FinishedExport(val uri: String) : ImportExportState()
-
-    data class ConfirmImport(val fileUri: String, val keyMapCount: Int) : ImportExportState()
-    data object Importing : ImportExportState()
-    data object FinishedImport : ImportExportState()
-    data class Error(val error: String) : ImportExportState()
-}
-
 sealed class HomeState {
     data class Selecting(
         val selectionCount: Int,