Merge pull request #175 from YAPP-Github/BOOK-304-feature/#173

feat: UX 개선을 위한 LoadingIndicator에 Delay 옵션 추가

Author: seoyoon513

core/ocr/build.gradle.kts

@@ -26,9 +26,6 @@ dependencies {
         projects.core.common,
 
         libs.logger,
-        libs.androidx.camera.core,
-
-        libs.google.mlkit.text.recognition.korean,
     )
 }
 

core/ocr/src/main/kotlin/com/ninecraft/booket/core/ocr/analyzer/LiveTextAnalyzer.kt

@@ -1,4 +1,4 @@
-package com.ninecraft.booket.core.ocr.analyzer
+package com.ninecraft.booket.core.ocr.recognizer
 
 import android.net.Uri
 import android.util.Base64

core/ocr/src/main/kotlin/com/ninecraft/booket/core/ocr/analyzer/StillTextAnalyzer.kt

@@ -4,23 +4,34 @@ import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.material3.CircularProgressIndicator
 import androidx.compose.runtime.Composable
+import androidx.compose.runtime.getValue
+import androidx.compose.runtime.produceState
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import com.ninecraft.booket.core.common.extensions.noRippleClickable
 import com.ninecraft.booket.core.designsystem.ComponentPreview
 import com.ninecraft.booket.core.designsystem.theme.ReedTheme
+import kotlinx.coroutines.delay
 
 @Composable
 fun ReedLoadingIndicator(
     modifier: Modifier = Modifier,
+    delayMillis: Long = 500L,
 ) {
-    Box(
-        modifier = modifier
-            .fillMaxSize()
-            .noRippleClickable {},
-        contentAlignment = Alignment.Center,
-    ) {
-        CircularProgressIndicator(color = ReedTheme.colors.contentBrand)
+    val showProgressBar by produceState(initialValue = false, key1 = delayMillis) {
+        delay(delayMillis)
+        value = true
+    }
+
+    if (showProgressBar) {
+        Box(
+            modifier = modifier
+                .fillMaxSize()
+                .noRippleClickable {},
+            contentAlignment = Alignment.Center,
+        ) {
+            CircularProgressIndicator(color = ReedTheme.colors.contentBrand)
+        }
     }
 }
 

core/ocr/src/main/kotlin/com/ninecraft/booket/core/ocr/analyzer/TextAnalyzer.kt

@@ -22,7 +22,6 @@ import com.ninecraft.booket.feature.screens.RecordDetailScreen
 import com.ninecraft.booket.feature.screens.RecordEditScreen
 import com.ninecraft.booket.feature.screens.RecordScreen
 import com.ninecraft.booket.feature.screens.arguments.RecordEditArgs
-import com.ninecraft.booket.feature.screens.extensions.delayedGoTo
 import com.orhanobut.logger.Logger
 import com.skydoves.compose.effects.RememberedEffect
 import com.slack.circuit.codegen.annotations.CircuitInject
@@ -304,15 +303,13 @@ class BookDetailPresenter @AssistedInject constructor(
 
                 is BookDetailUiEvent.OnShareRecordClick -> {
                     isRecordMenuBottomSheetVisible = false
-                    scope.launch {
-                        navigator.delayedGoTo(
-                            RecordCardScreen(
-                                quote = selectedRecordInfo.quote,
-                                bookTitle = selectedRecordInfo.bookTitle,
-                                emotionTag = selectedRecordInfo.emotionTags[0],
-                            ),
-                        )
-                    }
+                    navigator.goTo(
+                        RecordCardScreen(
+                            quote = selectedRecordInfo.quote,
+                            bookTitle = selectedRecordInfo.bookTitle,
+                            emotionTag = selectedRecordInfo.emotionTags[0],
+                        ),
+                    )
                 }
 
                 is BookDetailUiEvent.OnEditRecordClick -> {

core/ocr/src/main/kotlin/com/ninecraft/booket/core/ocr/di/OcrModule.kt

@@ -136,7 +136,10 @@ internal fun BookDetailUi(
             },
             sheetState = recordMenuBottomSheetState,
             onShareRecordClick = {
-                state.eventSink(BookDetailUiEvent.OnShareRecordClick)
+                coroutineScope.launch {
+                    recordMenuBottomSheetState.hide()
+                    state.eventSink(BookDetailUiEvent.OnShareRecordClick)
+                }
             },
             onEditRecordClick = {
                 coroutineScope.launch {

…/core/ocr/analyzer/CloudOcrRecognizer.kt …ore/ocr/recognizer/CloudOcrRecognizer.ktcore/ocr/src/main/kotlin/com/ninecraft/booket/core/ocr/analyzer/CloudOcrRecognizer.kt renamed to core/ocr/src/main/kotlin/com/ninecraft/booket/core/ocr/recognizer/CloudOcrRecognizer.kt core/ocr/src/main/kotlin/com/ninecraft/booket/core/ocr/analyzer/CloudOcrRecognizer.kt renamed to core/ocr/src/main/kotlin/com/ninecraft/booket/core/ocr/recognizer/CloudOcrRecognizer.kt

@@ -14,7 +14,6 @@ import com.ninecraft.booket.feature.screens.RecordCardScreen
 import com.ninecraft.booket.feature.screens.RecordDetailScreen
 import com.ninecraft.booket.feature.screens.RecordEditScreen
 import com.ninecraft.booket.feature.screens.arguments.RecordEditArgs
-import com.ninecraft.booket.feature.screens.extensions.delayedGoTo
 import com.orhanobut.logger.Logger
 import com.skydoves.compose.effects.RememberedEffect
 import com.slack.circuit.codegen.annotations.CircuitInject
@@ -124,15 +123,13 @@ class RecordDetailPresenter @AssistedInject constructor(
 
                 is RecordDetailUiEvent.OnShareRecordClick -> {
                     isRecordMenuBottomSheetVisible = false
-                    scope.launch {
-                        navigator.delayedGoTo(
-                            RecordCardScreen(
-                                quote = recordDetailInfo.quote,
-                                bookTitle = recordDetailInfo.bookTitle,
-                                emotionTag = recordDetailInfo.emotionTags[0],
-                            ),
-                        )
-                    }
+                    navigator.goTo(
+                        RecordCardScreen(
+                            quote = recordDetailInfo.quote,
+                            bookTitle = recordDetailInfo.bookTitle,
+                            emotionTag = recordDetailInfo.emotionTags[0],
+                        ),
+                    )
                 }
 
                 is RecordDetailUiEvent.OnEditRecordClick -> {