better coroutine handling

Author: Raival-e

app/src/main/java/com/raival/compose/file/explorer/screen/main/tab/files/ui/FilesList.kt

@@ -73,6 +73,7 @@ import com.raival.compose.file.explorer.screen.preferences.constant.FileItemSize
 import com.raival.compose.file.explorer.screen.preferences.constant.FileItemSizeMap.getFileListIconSize
 import com.raival.compose.file.explorer.screen.preferences.constant.FileItemSizeMap.getFileListSpace
 import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.Dispatchers.IO
 import kotlinx.coroutines.delay
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
@@ -577,39 +578,36 @@ private fun FileDetails(
     fontSize: Int,
     isHighlighted: Boolean
 ) {
-    val scope = rememberCoroutineScope()
-    Isolate {
-        var details by remember(
-            key1 = currentItemPath,
-            key2 = item.lastModified
-        ) { mutableStateOf(emptyString) }
-
-        LaunchedEffect(
-            key1 = currentItemPath,
-            key2 = item.lastModified
-        ) {
-            scope.launch(Dispatchers.IO) {
-                if (details.isEmpty()) {
-                    val det = item.getDetails()
-                    details = det
-                }
+    var details by remember(
+        key1 = currentItemPath,
+        key2 = item.lastModified
+    ) { mutableStateOf(emptyString) }
+
+    LaunchedEffect(
+        key1 = currentItemPath,
+        key2 = item.lastModified
+    ) {
+        if (details.isEmpty()) {
+            val det = withContext(IO) {
+                item.getDetails()
             }
+            details = det
         }
-
-        Text(
-            modifier = Modifier.alpha(0.7f),
-            text = details,
-            fontSize = (fontSize - 4).sp,
-            maxLines = 1,
-            lineHeight = (fontSize + 2).sp,
-            overflow = TextOverflow.Ellipsis,
-            color = if (isHighlighted) {
-                colorScheme.primary
-            } else {
-                Color.Unspecified
-            }
-        )
     }
+
+    Text(
+        modifier = Modifier.alpha(0.7f),
+        text = details,
+        fontSize = (fontSize - 4).sp,
+        maxLines = 1,
+        lineHeight = (fontSize + 2).sp,
+        overflow = TextOverflow.Ellipsis,
+        color = if (isHighlighted) {
+            colorScheme.primary
+        } else {
+            Color.Unspecified
+        }
+    )
 }
 
 @Composable

app/src/main/java/com/raival/compose/file/explorer/screen/main/tab/files/ui/dialog/BookmarksDialog.kt

@@ -45,7 +45,8 @@ import com.raival.compose.file.explorer.common.ui.Space
 import com.raival.compose.file.explorer.screen.main.tab.files.FilesTab
 import com.raival.compose.file.explorer.screen.main.tab.files.holder.LocalFileHolder
 import com.raival.compose.file.explorer.screen.main.tab.files.ui.FileItemRow
-import kotlinx.coroutines.launch
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import java.io.File
 
 @OptIn(ExperimentalMaterialApi::class, ExperimentalFoundationApi::class)
@@ -62,7 +63,7 @@ fun BookmarksDialog(
         }
 
         LaunchedEffect(Unit) {
-            tab.scope.launch {
+            withContext(Dispatchers.IO) {
                 val originalList = globalClass.preferencesManager.bookmarks
                 bookmarks.addAll(
                     originalList.map { LocalFileHolder(File(it)) }.filter { it.isValid() }

app/src/main/java/com/raival/compose/file/explorer/screen/main/tab/home/ui/HomeLayoutSettingsScreen.kt

@@ -64,7 +64,7 @@ import com.raival.compose.file.explorer.screen.main.tab.home.data.HomeSectionCon
 import com.raival.compose.file.explorer.screen.main.tab.home.data.HomeSectionType
 import com.raival.compose.file.explorer.screen.main.tab.home.data.getDefaultHomeLayout
 import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
 import sh.calvin.reorderable.ReorderableCollectionItemScope
 import sh.calvin.reorderable.ReorderableItem
 import sh.calvin.reorderable.rememberReorderableLazyListState
@@ -103,7 +103,7 @@ fun HomeLayoutSettingsScreen(
         }
 
         LaunchedEffect(Unit) {
-            scope.launch(Dispatchers.IO) {
+            withContext(Dispatchers.IO) {
                 val config = try {
                     Gson().fromJson(
                         globalClass.preferencesManager.homeTabLayout,

app/src/main/java/com/raival/compose/file/explorer/screen/main/tab/home/ui/HomeTabContentView.kt

@@ -82,6 +82,7 @@ import com.raival.compose.file.explorer.screen.main.ui.StorageDeviceView
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.async
 import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
 
 @OptIn(ExperimentalFoundationApi::class)
 @Composable
@@ -91,7 +92,7 @@ fun ColumnScope.HomeTabContentView(tab: HomeTab) {
     val enabledSections = remember { mutableStateListOf<HomeSectionConfig>() }
 
     LaunchedEffect(tab.id) {
-        scope.launch(Dispatchers.IO) {
+        withContext(Dispatchers.IO) {
             async {
                 tab.fetchRecentFiles()
             }

app/src/main/java/com/raival/compose/file/explorer/screen/viewer/pdf/ui/PdfViewerContent.kt

@@ -20,7 +20,6 @@ import androidx.compose.runtime.getValue
 import androidx.compose.runtime.mutableStateListOf
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
-import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.setValue
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
@@ -31,7 +30,7 @@ import com.raival.compose.file.explorer.common.isNot
 import com.raival.compose.file.explorer.screen.viewer.pdf.PdfViewerInstance
 import com.raival.compose.file.explorer.screen.viewer.pdf.misc.PdfPageHolder
 import kotlinx.coroutines.Dispatchers.IO
-import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
 import my.nanihadesuka.compose.InternalLazyColumnScrollbar
 import my.nanihadesuka.compose.ScrollbarLayoutSide
 import my.nanihadesuka.compose.ScrollbarSettings
@@ -61,7 +60,6 @@ fun PdfViewerContent(instance: PdfViewerInstance, onBackPress: () -> Unit) {
                 listState.firstVisibleItemIndex == 0
             }
         }
-        val scope = rememberCoroutineScope()
 
         val visiblePageNumbers by remember {
             derivedStateOf {
@@ -83,7 +81,7 @@ fun PdfViewerContent(instance: PdfViewerInstance, onBackPress: () -> Unit) {
         }
 
         LaunchedEffect(Unit) {
-            scope.launch(IO) {
+            withContext(IO) {
                 instance.prepare { success ->
                     if (success) {
                         defaultPageSize = Size(