홈과 보관함 폴더 생성/이름 변경 UI 및 navigation 추가 #67

ddyeon · 2024-07-07T09:22:31Z

개요

이슈 링크 혹은 PR 내용 요약

홈과 보관함에서 사용되는 "새폴더 추가"와 "폴더 이름 변경" UI 만들었습니다.

작업 내용

실제 작업 내용

홈과 보관함 페이지가 비슷하지만 각각 화면에서 사용하여 따로 만들어주었습니당~~
(묵주가 만들어둔 화면덕분에 잘 재활용 함ㅎㅎ 근데 alert는 API붙이면서 수정할게용)
홈화면 -> 바텀시트 -> 새 폴더 추가 누르면 나오고
보관함 -> 맨위 topbar나 각 아이템 맨오른쪾 아이콘 누르면 이동합니당

시연 화면 (option)

실행 스크린샷 혹은 영상 첨부

새폴더추가	폴더이름변경

To Reviers

리뷰어들에게 전할 말

현재 다이얼로그 뜰때 딤처리 안되는데 제가 수정해보겠습니당~
폴더생성 API는 오늘 밤 계속 될 예정..coming,,soon,,

Close

close #60

# Conflicts: # app/src/main/java/com/mashup/dorabangs/navigation/MainNavHost.kt # feature/home/src/main/java/com/mashup/dorabangs/feature/home/HomeScreen.kt # feature/home/src/main/java/com/mashup/dorabangs/feature/navigation/HomeNavigation.kt

...ure/home/src/main/java/com/mashup/dorabangs/feature/navigation/HomeCreateFolderNavigation.kt

Ahn-seokjoo · 2024-07-07T11:25:21Z

feature/storage/src/main/java/com/mashup/dorabangs/feature/navigation/FolderManageNavigation.kt

+        val folderManageType = navBackStackEntry.arguments?.getString("folderManageType") ?: FolderManageType.CHANGE.name
+        StorageFolderManageRoute(
+            folderManageType = folderManageType,
+            onClickBackIcon = onClickBackIcon,


여기부터 name으로 주고 있넹 type으로 못주나
여기서 name -> 뷰모델에서 타입으로 넣던뎅

흠 나도 이부분 고민됐는데 navArgument 받으려면
type = NavType.StringType 요부분때매 string으로 받았거든ㅠㅠ
좀 더 방법을 생각해보겟쑴당🫡

@Ahn-seokjoo @ddyeon
커스텀 타입을 전달할 수 있는 방법을 물어보는건가 ??
Compose Navigation에 safeArg가 있긴 해

호오 커스텀 화긴~~해볼게용

굿굿 저런식으로 하면 될듯~ 상점 1점 드립니다

ddyeon · 2024-07-08T15:08:16Z

디테일 조금 더 잡아서 수정 올려쒀요 묵주 호묵 확인부탁

fbghgus123

@ddyeon
고생했어 다묵이

fbghgus123 · 2024-07-09T00:18:06Z

...rc/main/java/com/mashup/dorabangs/core/designsystem/component/bottomsheet/BottomSheetType.kt

+        firstItemName: Int,
+        secondItemName: Int,


@ddyeon
리소스 어노테이션 붙이면 더 좋을듯 ~~

fbghgus123 · 2024-07-09T00:24:58Z

feature/home/src/main/java/com/mashup/dorabangs/feature/home/HomeScreen.kt

+
+    LaunchedEffect(key1 = Unit) {
+        if (isVisibleBottomSheet) {
+            viewModel.setVisibleMovingFolderBottomSheet(true)
+        }
+    }
+


@ddyeon
viewModel에 있는 데이터를 업데이트 하는 것 보다는 ViewModel의 SavedStateHandle에서 데이터를 받는게 더 좋아보여 ~
hiltViewModel이 그런거 해주는 녀석일거야

fbghgus123 · 2024-07-09T00:27:45Z

feature/home/src/main/java/com/mashup/dorabangs/feature/navigation/HomeNavigation.kt

+        val isVisibleBottomSheet = backStackEntry.savedStateHandle.getStateFlow(
+            "isVisibleMovingBottomSheet",
+            initialValue = false,
+        ).collectAsState().value


@ddyeon
flow로 넘기고 collet해서 받는 방법이 필요한 이유가 뭐야 ??
그냥 Boolean 값을 가져오면 안되나?

없는 것 같습니다. viewmodel에서 collect할 일이 없을 것 같아소 get으로 변경하겠숨당~

fbghgus123 · 2024-07-09T00:28:53Z

feature/storage/src/main/java/com/mashup/dorabangs/feature/folders/StorageFolderManageScreen.kt

+    LaunchedEffect(key1 = Unit) {
+        folderManageViewModel.setFolderManageType(folderManageType)
+    }


@ddyeon
이부분도 savedStateHandle에서 바로 받는게 좋을듯 ??

initial 부분이라 savedStateHandle로 주자는거야 ? ? 아님 저런 set get ㅎ함수를 사용하지 말자? @fbghgus123

@Ahn-seokjoo
어차피 ViewModel에서 지지고 볶고 할 데이터를 굳이 Compose 단까지 가져오지 말자는 말.
ViewModel을 hiltViewModel()로 만들면 SavedStateHandle에서 백 스택 엔트리에 저장된 데이터를 가져올 수 잇음 ~~

음. 저 LaunchEffect 용도가 initial data set해주는 용도같아서 나도 무러본거였엉
initialSet이면 호현말대루 savedstatehandle로 받음 편한뎅 그냥 다른곳에서ㅏ도 set해주려는거면 저 set함수 자체는 필요하지 않나 싶어서 무러봤슴

initialize가 목적이라면 나도 찬성이야~

fbghgus123 · 2024-07-09T00:29:46Z

feature/storage/src/main/java/com/mashup/dorabangs/feature/folders/model/FolderManageType.kt

+enum class FolderManageType(@StringRes val title: Int) {
+    CREATE(R.string.storage_create_folder_title),
+    CHANGE(R.string.storage_edit_folder_title),
+}


@ddyeon
이거는 무엇을 위한 구분이야?

폴더 생성과 폴더 이름 변경을 같은 페이지로 쓰고 있어서 tilte만 바꿔주는용도!!

fbghgus123 · 2024-07-09T00:32:50Z

.../storage/src/main/java/com/mashup/dorabangs/feature/storage/storage/StorageFolderListItem.kt

@@ -152,7 +152,7 @@ fun StorageListItem(
                    androidx.core.R.drawable.ic_call_answer
                }


@ddyeon
미리 분기 처리해놓은거 좋앗다 ??

fbghgus123 · 2024-07-09T00:33:31Z

feature/storage/src/main/java/com/mashup/dorabangs/feature/navigation/StorageNavigation.kt

+    navigateToStorageDetail: (StorageFolderItem) -> Unit = {},
+    navigateToFolderManage: (FolderManageType) -> Unit,


@ddyeon
디폴트가 있고 없고의 차이는 뭐야 ??
아래는 필수라는건가?

저는 보통 프리뷰때매 적는편인뎅,,, 묵주는 안적는 것 같고,, 그래서 슬며시 안적어봤는데,,,
통일하는게좋겟쥬?

프리뷰야뭐 {} 넣어주면 되는데, 사용할때 누구나 안에까지 본다는 가정을 하면 안될 것 같아서 난 반드시 필요한 건 default 안넣긴행

필수이면 안넣는게 맞긴 해 ~~

fbghgus123 · 2024-07-09T00:36:20Z

feature/storage/src/main/java/com/mashup/dorabangs/feature/navigation/FolderManageNavigation.kt

+        val folderManageType = navBackStackEntry.arguments?.getString("folderManageType") ?: FolderManageType.CHANGE.name
+        StorageFolderManageRoute(
+            folderManageType = folderManageType,
+            onClickBackIcon = onClickBackIcon,


@Ahn-seokjoo @ddyeon
커스텀 타입을 전달할 수 있는 방법을 물어보는건가 ??
Compose Navigation에 safeArg가 있긴 해

ddyeon · 2024-07-10T16:22:54Z

...ure/home/src/main/java/com/mashup/dorabangs/feature/navigation/HomeCreateFolderNavigation.kt

+fun NavGraphBuilder.homeCreateFolderNavigation(
+    navController: NavController,
+    onClickBackIcon: () -> Unit,
+) {
+    composable(
+        route = NavigationRoute.HomeScreen.HomeCreateFolder.route,
+    ) {
+        navController.currentBackStackEntry?.let { backStackEntry ->
+            val homeViewModel: HomeViewModel = hiltViewModel(backStackEntry)
+            homeViewModel.savedStateHandle["isVisibleMovingBottomSheet"] = true
+


여기서 결국은 Navcontroller.previousBackStackEntry을 사용하는거여서 MainNavHost에서 지금 navcontroller받고있는데 이걸 previousBackStackEntry 넘기는 걸로 바뀌면 못찾네용,,???!!!

그래서 일단은 navcontroller 받는걸로 했는뎀 불만 있는 사람 이유 찾아죠ㅠㅠㅎ
@fbghgus123 @Ahn-seokjoo

ㅋㅋㅋ 일단 난 뭔 소린지 이해 못했어.

ddyeon added 11 commits July 7, 2024 16:48

[feat] 홈화면 폴더 생성 화면 추가

98d27bd

[feat] 폴더생성 navigation 생성

0f2dcb1

[feat] 보관함쪽 폴더 생성 UI 추가

59a9d3b

[feat] 보관함 폴더 생성 네비게이션 연결

f7d407a

[feat] 필요한 string 추가

4f4141e

[feat] 폴더관리 모델 및 타입 생성

5a5d642

[feat] 보관함 dilaog, bottomsheet추가

47f7320

[feat] 보관함 dilaog, bottomsheet추가

b0e0a0e

[feat] 보관함에서 폴더관리 navigation 세팅

b39c6f8

[fix] spotless 적용

637e0e6

ddyeon requested review from Ahn-seokjoo and fbghgus123 as code owners July 7, 2024 09:22

Merge branch 'develop' into feature/create-folder-ui

bcdb5b8

ddyeon self-assigned this Jul 7, 2024

ddyeon added the feature develop feature label Jul 7, 2024

Ahn-seokjoo reviewed Jul 7, 2024

View reviewed changes

...ure/home/src/main/java/com/mashup/dorabangs/feature/navigation/HomeCreateFolderNavigation.kt Outdated Show resolved Hide resolved

Ahn-seokjoo reviewed Jul 7, 2024

View reviewed changes

Ahn-seokjoo approved these changes Jul 7, 2024

View reviewed changes

ddyeon added 3 commits July 7, 2024 22:15

[fix] 홈 폴더생성 뒤로가기 추가

fd66fbc

[fix] bottomsheet 닫힌 후 이동하도록 수정

2f80adf

[fix] 폴더 생성 페이지에서 뒤로가기 시 바텀 시트 노출되도록 수정

776d6b6

fbghgus123 approved these changes Jul 9, 2024

View reviewed changes

ddyeon added 6 commits July 9, 2024 22:07

[fix] bottomsheet annotation 추가

b087a47

[fix] defaultvalue제거

c749480

[fix] savestatehandler로 값 받기 수정

2e35cb8

[fix] savestatehandle 찾는중,,

af5835a

[fix] savestatehandle을 위한,, viewmodel공유

ff0bd1a

[fix] savestatehandle을 위한,, viewmodel공유

bb8d8ea

ddyeon added 2 commits July 11, 2024 01:19

[fix] bottomsheet 변경

a8dac49

[fix] 홈 폴더 생성 naventry 변경

66093bb

ddyeon commented Jul 10, 2024

View reviewed changes

ddyeon merged commit fa64dd3 into develop Jul 11, 2024
2 checks passed

ddyeon deleted the feature/create-folder-ui branch July 14, 2024 13:27

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

홈과 보관함 폴더 생성/이름 변경 UI 및 navigation 추가 #67

홈과 보관함 폴더 생성/이름 변경 UI 및 navigation 추가 #67

ddyeon commented Jul 7, 2024

Ahn-seokjoo Jul 7, 2024

ddyeon Jul 7, 2024

fbghgus123 Jul 9, 2024

ddyeon Jul 9, 2024

Ahn-seokjoo Jul 9, 2024

ddyeon commented Jul 8, 2024

fbghgus123 left a comment

fbghgus123 Jul 9, 2024

fbghgus123 Jul 9, 2024

fbghgus123 Jul 9, 2024

ddyeon Jul 9, 2024

fbghgus123 Jul 9, 2024

Ahn-seokjoo Jul 9, 2024

fbghgus123 Jul 9, 2024 •

edited

Loading

Ahn-seokjoo Jul 9, 2024

fbghgus123 Jul 9, 2024

ddyeon Jul 9, 2024

fbghgus123 Jul 9, 2024

fbghgus123 Jul 9, 2024

ddyeon Jul 9, 2024

Ahn-seokjoo Jul 9, 2024

fbghgus123 Jul 9, 2024

fbghgus123 Jul 9, 2024

ddyeon Jul 10, 2024

Ahn-seokjoo Jul 11, 2024

		@@ -152,7 +152,7 @@ fun StorageListItem(
		androidx.core.R.drawable.ic_call_answer
		}

		navigateToStorageDetail: (StorageFolderItem) -> Unit = {},
		navigateToFolderManage: (FolderManageType) -> Unit,

홈과 보관함 폴더 생성/이름 변경 UI 및 navigation 추가 #67

홈과 보관함 폴더 생성/이름 변경 UI 및 navigation 추가 #67

Conversation

ddyeon commented Jul 7, 2024

개요

작업 내용

시연 화면 (option)

To Reviers

Close

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ddyeon commented Jul 8, 2024

fbghgus123 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

fbghgus123 Jul 9, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

fbghgus123 Jul 9, 2024 •

edited

Loading