[init] Jetpack Navigation 세팅 #10

Hyobeen-Park · 2024-05-15T04:52:46Z

Related issue 🛠

closed [init] Jetpack Navigation 세팅 #8

Work Description ✏️

navigation 세팅했습니다.
MainActivity 만들었습니다.
fragment들도 미리 만들어뒀어요!
Bottom Navigation 만들었습니다(fragment에 따라 visibility도 설정되도록 했습니다!)

Screenshot 📸

Uncompleted Tasks 😅

N/A

To Reviewers 📢

Fragment 네이밍이랑 패키지 구조 괜찮은지 한번만 확인해주세요!!
MainActivity를 만들어서 이제 DummyActicity를 안쓰는데 이거는 지금 바로 없애는게 나을까요..? 연결되어 있는게 많은 것 같아서 두긴 했는데...🥲

…ty-architecture

jihyunniiii

DummyActivity는 일단 그대로 두는 걸로 합시당 서버통신 할 때 서버통신 관련 더미 파일이 필요할 것 같아서요 ~ 마지막 날 뷰 연결 작업 하면서 지우는 걸로 합시다 ~
고생하셨습니다 ~

jihyunniiii · 2024-05-15T04:56:37Z

app/build.gradle.kts

+    // Jetpack Navigation
+    implementation(libs.androidx.navigation.fragment)
+    implementation(libs.androidx.navigation.ui)


androidx 번들에 넣어서 미리 추가해두었어요! 제거해주셔도 됩니다 ~

아하아하!! 이해했어요!!! 요건 제거하겠습니다~!

jihyunniiii · 2024-05-15T05:16:33Z

app/src/main/AndroidManifest.xml

패키지 구조 잘 짜셨는데요??
근데 조금 더 확장성을 고려해보시면 좋을 것 같아요! 지금은 구현하는 뷰가 4개지만, 크림 내부의 모든 화면을 구현한다고 생각하면 지금 main 화면 내에 바텀 네비가 있어서 home, style, shop, saved, my의 5개의 프래그먼트로 전환이 가능하고, home 프래그먼트 내부에서도 탭 레이아웃이 있어서 추천, 랭킹, 발매정도 등으로 전환이 가능한 구조잖아요! 그래서 저라면 main 내에 home 패키지를 하나 더 만들고 그 내부에 recommend와 release 패키지를 위치 시킬 것 같아요!

오호 그렇네요!! 그럼 home 패키지 하나 더 추가하겠습니다!! 감사해요😊

jihyunniiii · 2024-05-15T05:17:35Z

app/src/main/java/org/sopt/kream/presentation/ui/main/MainActivity.kt

+import androidx.appcompat.app.AppCompatActivity
+import org.sopt.kream.databinding.ActivityMainBinding
+
+class MainActivity : AppCompatActivity() {


Util에 Binding Activity와 BindingFragment 등을 만들어두었으니 적용해주시면 좋을 것 같습니다 ~~

사용 방법은 제 과제 레포 보시면 좋을 것 같아요 ~

와우,,,, 이거 진짜 신세계네요,, 진짜 최고다,,,,⭐

jihyunniiii · 2024-05-15T05:20:00Z

app/src/main/java/org/sopt/kream/presentation/ui/productDetail/ProductDetailFragment.kt

요기 패키지 productdetail로 수정 부탁드려용 (패키지 이름은 전부 소문자로!!)

아하 감사합니다!!

jihyunniiii · 2024-05-15T05:20:19Z

app/src/main/java/org/sopt/kream/presentation/ui/main/release/ReleaseFragment.kt

+    @Composable
+    fun releaseScreen() {
+        Text(text = "Release")
+    }


와 이 뷰는 컴포즈로 짠다고 해서 이렇게 넣어준 건가요? 센스 미쳤따

jihyunniiii · 2024-05-15T05:23:29Z

app/src/main/res/layout/activity_main.xml

바텀 네비게이션은 다른 PR에서 구현하실 예정이신가용? 이 피알에서 같이 해도 좋을 것 같은데!
바텀 네비게이션은 메인에 구현을 해두고 fcv에 띄워주는 fragment에 따라 visibility를 조절해주면 됩니다 ~

따로 해야 하는줄 알고 안했는데..!! 여기다가 바로 할게용😊 바텀 네비게이션 어떻게 해야하나 고민했는데 visibility로 조절하면 되는군요!!! 감사합니다ㅎㅎㅎ

jihyunniiii · 2024-05-15T05:27:29Z

app/src/main/res/navigation/nav_graph.xml

+        android:label="fragment_recommend"
+        tools:layout="@layout/fragment_recommend">
+        <action
+            android:id="@+id/action_recommendFragment_to_productDetailFragment"


Suggested change

android:id="@+id/action_recommendFragment_to_productDetailFragment"

android:id="@+id/action_recommend_fragment_to_product_detail_fragment"

이런 식으로 네이밍 형태를 유지해주시는 게 좋을 것 같아요 ~

오호 수정하겠습니다~!!

jihyunniiii · 2024-05-15T05:32:52Z

app/src/main/res/navigation/nav_graph.xml

tablayout으로 이동되는 부분은 연결을 안 해도 될 것 같은데! 사실 저도 jetpackNavi를 사용해본 적이 거의 없어서 확실하지 않네요 ㅜㅜ 하면서 수정해보면 좋을 것 같아요 ㅜ
그리고 액션이 생각보다 많아서 고민해보니까 main에서 바텀 네비를 비지빌리티를 통해 조절하는 것처럼 상단 검색 + 탭도 비지빌리티를 통해 조절하는 것도 좋아 보입니다 여러 뷰에 공통되는 부분이기도 하고 실제 앱을 보니까 하단 바텀 네비가 바뀌더라도 위에 검색 + 탭이 있는 형식은 유지가 되더라고요~ 공통 컴포넌트 구현하실 때 이 부분 생각해보시고 구현하시면 더 효율적이게 뷰를 짤 수 있을 것 같아요 ~

@Hyobeen-Park @t1nm1ksun

저도 action이 너무 많아서 고민했었는데ㅜㅜ 연결 없이 해보고 성공하면 수정하도록 하겠습니다!! 공통 컴포넌트도 visibility 사용하는건 생각 못했었는데!! 좋은 방법인 것 같아요 조언 감사합니다❤️❤️

…ty-architecture

jihyunniiii

SAA 설정 처음이라 쉽지 않으셨을텐데 정말 잘 하셨네요!
네이밍만 조금 수정 부탁드릴게요 ㅜㅜ

jihyunniiii · 2024-05-15T18:30:39Z

app/src/main/java/org/sopt/kream/presentation/ui/main/MainActivity.kt

+        val navHostFragment = supportFragmentManager.findFragmentById(R.id.fcv_main) as NavHostFragment
+        navController = navHostFragment.findNavController()
+
+        navController.addOnDestinationChangedListener { _, destination, _ ->
+            if (destination.id == R.id.recommend_fragment || destination.id == R.id.release_fragment) {
+                binding.bnvMain.visibility = View.VISIBLE
+            } else if (destination.id == R.id.search_fragment || destination.id == R.id.product_detail_fragment) {
+                binding.bnvMain.visibility = View.GONE
+            }
+        }


onCreate 내부에는 함수만 들어가는 것이 좋습니다 적절하게 함수화 해보시면 좋을 것 같아요 ~

아하 전부 다 함수로 빼야하는군요!! 수정하겠습니다!!

jihyunniiii · 2024-05-15T18:31:15Z

app/src/main/java/org/sopt/kream/presentation/ui/main/MainActivity.kt

+import org.sopt.kream.util.base.BindingActivity
+
+class MainActivity : BindingActivity<ActivityMainBinding>({ ActivityMainBinding.inflate(it) }) {
+    private lateinit var navController: NavController


전역으로 설정할 필요가 없을 것 같아요!

흠 그러네요..😅 감사합니다!!

jihyunniiii · 2024-05-15T18:31:46Z

app/src/main/java/org/sopt/kream/presentation/ui/main/MainActivity.kt

+        navController = navHostFragment.findNavController()
+
+        navController.addOnDestinationChangedListener { _, destination, _ ->
+            if (destination.id == R.id.recommend_fragment || destination.id == R.id.release_fragment) {


list를 만들고 in을 사용해 검사를 해봐도 좋을 것 같네요

이거 안그래도 더 좋은 방법이 없을까 고민했었는데...!! 감사합니다❤️❤️

jihyunniiii · 2024-05-15T18:31:55Z

app/src/main/java/org/sopt/kream/presentation/ui/main/MainActivity.kt

+        navController.addOnDestinationChangedListener { _, destination, _ ->
+            if (destination.id == R.id.recommend_fragment || destination.id == R.id.release_fragment) {
+                binding.bnvMain.visibility = View.VISIBLE
+            } else if (destination.id == R.id.search_fragment || destination.id == R.id.product_detail_fragment) {


그냥 else 해주셔도 됩니당

jihyunniiii · 2024-05-15T18:33:32Z

app/src/main/java/org/sopt/kream/presentation/ui/main/home/recommend/RecommendFragment.kt

+        binding.btnToProductDetail.setOnClickListener {
+            findNavController().navigate(R.id.release_fragment)
+        }


여기도 다 함수화 해주세요

jihyunniiii · 2024-05-15T18:35:38Z

app/src/main/res/layout/activity_main.xml

+        android:id="@+id/bnv_main"
+        android:layout_width="0dp"
+        android:layout_height="55dp"
+        android:background="@drawable/shape_gray06_border_1"


bnv의 전체에 테두리가 있는 게 아니라 상단에만 구분선이 있는 형태니까 background보다는 view를 사용하는 것이 적절할 것 같습니다 ~

넵~!! view로 수정하겠습니다!!

jihyunniiii · 2024-05-15T18:39:19Z

app/src/main/res/layout/activity_main.xml

+        app:itemPaddingTop="9dp"
+        app:itemTextAppearanceActive="@style/TextAppearance.Kream.Body8.SemiBold"
+        app:itemTextAppearanceInactive="@style/TextAppearance.Kream.Body8.SemiBold"
+        app:itemTextColor="@drawable/selector_bnv_label"


color 관련 selector는 res내부에 color 패키지 만들어서 거기에 넣어주세요
그리고 selector 네이밍은 selector_어느뷰에서 사용되는지_어떤 selector인지를 따릅니당
여기서는 selector_main_bnv_item_text_color 정도가 되겠네요

우왕 감사합니다!!!!

jihyunniiii · 2024-05-15T18:40:08Z

app/src/main/res/layout/activity_main.xml

+
+    <com.google.android.material.bottomnavigation.BottomNavigationView
+        android:id="@+id/bnv_main"
+        android:layout_width="0dp"


0dp 사용 좋네요 ~

…ty-architecture # Conflicts: # app/src/main/java/org/sopt/kream/presentation/ui/dummy/DummyActivity.kt # app/src/main/java/org/sopt/kream/presentation/ui/dummy/View1Screen.kt # app/src/main/java/org/sopt/kream/theme/Type.kt # app/src/main/res/values/strings.xml

jihyunniiii

머지하세용 ~

Hyobeen-Park added 6 commits May 15, 2024 11:29

[add] #8 dependencies 추가

22db47e

[feat] #8 nav_graph 구현

ea9e067

[feat] #8 MainActivity 생성

3370709

[feat] #8 Fragment 연결

a334278

Merge remote-tracking branch 'origin/develop' into init-single-activi…

9fc9090

…ty-architecture

[chore] #8 코드 정리

0876df4

Hyobeen-Park added 🐶 효빈 🐶 ✨ init ✨ labels May 15, 2024

Hyobeen-Park added this to the 프로젝트 기초 세팅 milestone May 15, 2024

Hyobeen-Park requested review from jihyunniiii and t1nm1ksun May 15, 2024 04:52

jihyunniiii reviewed May 15, 2024

View reviewed changes

Hyobeen-Park added 3 commits May 15, 2024 17:24

[chore] #8 중복된 dependencies 삭제 및 패키지 구조 수정

3c818f0

[mod] #8 BindingActivity, BindingFragment 적용

f4a3fab

[chore] #8 nav_graph 네이밍 수정

9005bce

jihyunniiii assigned Hyobeen-Park May 15, 2024

Hyobeen-Park added 5 commits May 15, 2024 22:08

[feat] #8 바텀 네비게이션 메뉴 추가

57dc6f9

[feat] #8 바텀 네비게이션 UI 구현

2953f10

[feat] #8 현재 fragment에 따른 바텀 네비게이션 visibility 설정

fdd9e9d

[mod] #8 홈 아이콘 수정

14cf20d

Merge remote-tracking branch 'origin/develop' into init-single-activi…

452c89d

…ty-architecture

jihyunniiii reviewed May 15, 2024

View reviewed changes

Hyobeen-Park added 6 commits May 16, 2024 03:51

[chore] #8 코드 정리

3cca8b1

[mod] #8 바텀 네비게이션 visibility 함수화

75a36f1

[mod] #8 추천 뷰 버튼 클릭 리스너 함수화

e1d2955

[mod] #8 바텀네비게이션 테두리 View로 수정

efb1985

[chore] #8 selector 네이밍 수정

8d3b962

jihyunniiii approved these changes May 16, 2024

View reviewed changes

Hyobeen-Park merged commit d5c33a1 into develop May 16, 2024
1 check passed

Hyobeen-Park deleted the init-single-activity-architecture branch May 18, 2024 12:35

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[init] Jetpack Navigation 세팅 #10

[init] Jetpack Navigation 세팅 #10

Hyobeen-Park commented May 15, 2024 •

edited

Loading

jihyunniiii left a comment

jihyunniiii May 15, 2024

Hyobeen-Park May 15, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 15, 2024

jihyunniiii May 15, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 15, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 15, 2024

jihyunniiii May 15, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 15, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 15, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 15, 2024

jihyunniiii left a comment

jihyunniiii May 15, 2024

Hyobeen-Park May 16, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 16, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 16, 2024

jihyunniiii May 15, 2024

jihyunniiii May 15, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 16, 2024

jihyunniiii May 15, 2024

Hyobeen-Park May 16, 2024

jihyunniiii May 15, 2024

jihyunniiii left a comment

	android:id="@+id/action_recommendFragment_to_productDetailFragment"
	android:id="@+id/action_recommend_fragment_to_product_detail_fragment"

[init] Jetpack Navigation 세팅 #10

[init] Jetpack Navigation 세팅 #10

Conversation

Hyobeen-Park commented May 15, 2024 • edited Loading

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

jihyunniiii 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

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

jihyunniiii 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

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

jihyunniiii left a comment

Choose a reason for hiding this comment

Hyobeen-Park commented May 15, 2024 •

edited

Loading