refactor(PlaceDetailPreview): PlaceDetailPreviewSecondaryFragment Compose 마이그레이션

기존 View 시스템 기반의 `PlaceDetailPreviewSecondaryFragment`를 Jetpack Compose로 마이그레이션하여 UI 구현 방식을 변경했습니다.

- **`PlaceDetailPreviewSecondaryFragment.kt` 수정:**
    - `onViewCreated` 및 ViewBinding 관련 로직을 제거하고, `onCreateView`에서 `ComposeView`를 반환하도록 변경했습니다.
    - `PlaceDetailPreviewSecondaryScreen` 컴포저블을 사용하여 UI를 구성하고, `viewModel.selectedPlaceFlow`를 `collectAsStateWithLifecycle`로 구독하여 상태를 전달했습니다.
    - 뒤로 가기 콜백(`OnBackPressedCallback`) 활성화 로직을 `LaunchedEffect` 내에서 처리하도록 변경했습니다.
    - 클릭 시 발생하는 로그 기록 로직(`PlacePreviewClick`)을 컴포저블의 `onClick` 콜백으로 이동시켰습니다.

- **`PlaceDetailPreviewSecondaryScreen.kt` 수정:**
    - 컴포넌트 전체에 클릭 이벤트를 지원하기 위해 `onClick` 파라미터를 추가하고 `clickable` Modifier를 적용했습니다.
    - `PreviewAnimatableBox`에 `fillMaxWidth` Modifier와 `shape`(`radius2`) 설정을 추가하여 UI 레이아웃을 개선했습니다.

Author: oungsi2000

app/src/main/java/com/daedan/festabook/presentation/placeMap/placeDetailPreview/PlaceDetailPreviewSecondaryFragment.kt

@@ -1,26 +1,36 @@
 package com.daedan.festabook.presentation.placeMap.placeDetailPreview
 
 import android.os.Bundle
+import android.view.LayoutInflater
 import android.view.View
+import android.view.ViewGroup
 import androidx.activity.OnBackPressedCallback
+import androidx.compose.foundation.layout.Box
+import androidx.compose.foundation.layout.fillMaxSize
+import androidx.compose.foundation.layout.padding
+import androidx.compose.runtime.LaunchedEffect
+import androidx.compose.runtime.getValue
+import androidx.compose.ui.Alignment
+import androidx.compose.ui.Modifier
+import androidx.compose.ui.platform.ComposeView
+import androidx.compose.ui.platform.ViewCompositionStrategy
 import androidx.fragment.app.Fragment
 import androidx.fragment.app.viewModels
 import androidx.lifecycle.ViewModelProvider
-import coil3.load
+import androidx.lifecycle.compose.collectAsStateWithLifecycle
 import com.daedan.festabook.R
 import com.daedan.festabook.databinding.FragmentPlaceDetailPreviewSecondaryBinding
+import com.daedan.festabook.di.appGraph
 import com.daedan.festabook.di.fragment.FragmentKey
-import com.daedan.festabook.logging.logger
 import com.daedan.festabook.presentation.common.BaseFragment
 import com.daedan.festabook.presentation.common.OnMenuItemReClickListener
-import com.daedan.festabook.presentation.common.showBottomAnimation
 import com.daedan.festabook.presentation.common.showErrorSnackBar
-import com.daedan.festabook.presentation.placeDetail.model.PlaceDetailUiModel
 import com.daedan.festabook.presentation.placeMap.PlaceMapViewModel
 import com.daedan.festabook.presentation.placeMap.logging.PlacePreviewClick
 import com.daedan.festabook.presentation.placeMap.model.SelectedPlaceUiState
-import com.daedan.festabook.presentation.placeMap.model.getIconId
-import com.daedan.festabook.presentation.placeMap.model.getTextId
+import com.daedan.festabook.presentation.placeMap.placeDetailPreview.component.PlaceDetailPreviewSecondaryScreen
+import com.daedan.festabook.presentation.theme.FestabookTheme
+import com.daedan.festabook.presentation.theme.festabookSpacing
 import dev.zacsweers.metro.AppScope
 import dev.zacsweers.metro.ContributesIntoMap
 import dev.zacsweers.metro.Inject
@@ -43,56 +53,62 @@ class PlaceDetailPreviewSecondaryFragment(
             }
         }
 
-    override fun onViewCreated(
-        view: View,
+    override fun onCreateView(
+        inflater: LayoutInflater,
+        container: ViewGroup?,
         savedInstanceState: Bundle?,
-    ) {
-        super.onViewCreated(view, savedInstanceState)
-        setUpObserver()
-        setUpBackPressedCallback()
-    }
-
-    override fun onMenuItemReClick() {
-        viewModel.unselectPlace()
-    }
+    ): View {
+        return ComposeView(requireContext()).apply {
+            setViewCompositionStrategy(ViewCompositionStrategy.DisposeOnViewTreeLifecycleDestroyed)
+            setContent {
+                val placeDetailUiState by viewModel.selectedPlaceFlow.collectAsStateWithLifecycle()
+                val visible = placeDetailUiState is SelectedPlaceUiState.Success
 
-    private fun setUpBackPressedCallback() {
-        requireActivity().onBackPressedDispatcher.addCallback(
-            viewLifecycleOwner,
-            backPressedCallback,
-        )
-    }
-
-    private fun setUpObserver() {
-        viewModel.selectedPlace.observe(viewLifecycleOwner) { selectedPlace ->
-            backPressedCallback.isEnabled = true
-            when (selectedPlace) {
-                is SelectedPlaceUiState.Success -> {
-                    binding.layoutSelectedPlace.visibility = View.VISIBLE
-                    binding.layoutSelectedPlace.showBottomAnimation()
-                    updateSelectedPlaceUi(selectedPlace.value)
-                    binding.logger.log(
-                        PlacePreviewClick(
-                            baseLogData = binding.logger.getBaseLogData(),
-                            placeName = selectedPlace.value.place.title ?: "undefined",
-                            timeTag = viewModel.selectedTimeTag.value?.name ?: "undefined",
-                            category = selectedPlace.value.place.category.name,
-                        ),
-                    )
+                LaunchedEffect(placeDetailUiState) {
+                    backPressedCallback.isEnabled = true
                 }
 
-                is SelectedPlaceUiState.Error -> showErrorSnackBar(selectedPlace.throwable)
-                is SelectedPlaceUiState.Loading -> Unit
-                is SelectedPlaceUiState.Empty -> backPressedCallback.isEnabled = false
+                FestabookTheme {
+                    Box(
+                        modifier = Modifier.fillMaxSize(),
+                        contentAlignment = Alignment.BottomCenter,
+                    ) {
+                        PlaceDetailPreviewSecondaryScreen(
+                            visible = visible,
+                            placeUiState = placeDetailUiState,
+                            modifier =
+                                Modifier
+                                    .padding(
+                                        vertical = festabookSpacing.paddingBody4,
+                                        horizontal = festabookSpacing.paddingScreenGutter,
+                                    ),
+                            onError = {
+                                showErrorSnackBar(it.throwable)
+                            },
+                            onEmpty = {
+                                backPressedCallback.isEnabled = false
+                            },
+                            onClick = {
+                                if (it !is SelectedPlaceUiState.Success) return@PlaceDetailPreviewSecondaryScreen
+                                appGraph.defaultFirebaseLogger.log(
+                                    PlacePreviewClick(
+                                        baseLogData = appGraph.defaultFirebaseLogger.getBaseLogData(),
+                                        placeName = it.value.place.title ?: "undefined",
+                                        timeTag =
+                                            viewModel.selectedTimeTag.value?.name
+                                                ?: "undefined",
+                                        category = it.value.place.category.name,
+                                    ),
+                                )
+                            },
+                        )
+                    }
+                }
             }
         }
     }
 
-    private fun updateSelectedPlaceUi(selectedPlace: PlaceDetailUiModel) {
-        with(binding) {
-            ivSecondaryCategoryItem.load(selectedPlace.place.category.getIconId())
-            tvSelectedPlaceTitle.text =
-                selectedPlace.place.title ?: getString(selectedPlace.place.category.getTextId())
-        }
+    override fun onMenuItemReClick() {
+        viewModel.unselectPlace()
     }
 }

app/src/main/java/com/daedan/festabook/presentation/placeMap/placeDetailPreview/component/PlaceDetailPreviewSecondaryScreen.kt

@@ -1,6 +1,8 @@
 package com.daedan.festabook.presentation.placeMap.placeDetailPreview.component
 
+import androidx.compose.foundation.clickable
 import androidx.compose.foundation.layout.Row
+import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.size
 import androidx.compose.material3.Icon
@@ -22,6 +24,7 @@ import com.daedan.festabook.presentation.placeMap.model.getIconId
 import com.daedan.festabook.presentation.placeMap.model.getTextId
 import com.daedan.festabook.presentation.theme.FestabookTheme
 import com.daedan.festabook.presentation.theme.FestabookTypography
+import com.daedan.festabook.presentation.theme.festabookShapes
 import com.daedan.festabook.presentation.theme.festabookSpacing
 
 @Composable
@@ -30,11 +33,18 @@ fun PlaceDetailPreviewSecondaryScreen(
     modifier: Modifier = Modifier,
     onError: (SelectedPlaceUiState.Error) -> Unit = {},
     onEmpty: () -> Unit = {},
+    onClick: (SelectedPlaceUiState) -> Unit = {},
     visible: Boolean = false,
 ) {
     PreviewAnimatableBox(
         visible = visible,
-        modifier = modifier,
+        modifier =
+            modifier
+                .fillMaxWidth()
+                .clickable {
+                    onClick(placeUiState)
+                },
+        shape = festabookShapes.radius2,
     ) {
         when (placeUiState) {
             is SelectedPlaceUiState.Loading -> Unit
@@ -96,7 +106,7 @@ private val FAKE_PLACE =
         id = 1,
         imageUrl = null,
         category = PlaceCategoryUiModel.TOILET,
-        title = "테스트테스트테스트테스트테스트테스트테스트테스트테스트테스트테스트테스트테스트",
+        title = "테스트테스",
         description = "https://onlyfor-me-blog.tistory.com/1190",
         location = null,
         isBookmarked = false,