[UI/#218] 온보딩 / UI 수정 #219

leeeyubin · 2024-09-03T23:13:43Z

closed [UI] 온보딩 / UI 수정 #218

⛳️ Work Description

필터링 설정 버튼 수정
데이트 피커 수정
파일 위치 이동 및 로직 수정
데이터 관련 파일 만들기

📸 Screenshot

Screen_Recording_20240904_073123_.terning.mp4

📢 To Reviewers

이번에 제 피알에서도 Grey375이 쓰여서 일단 넣어놨어요! 아린 언니 머지하면 지울게욤
데이트 피커 관련해서 전면 수정했어요! (기존 거는 앱이 터지는 경우가 너무 많았음)
이게 효빈언니랑 같이 써야 하는 거라 core 모듈에서 작성한 후, feature 모듈에서 참조하려고 했는데 rememberSnapFlingBehavior()에서 자꾸 에러가 나더라구요. 그래서 일단은 feautre 모듈 안에 넣어놨습니당,, 뭐가 문젠지 더 알아볼게요!
그리구 저번에 회의 때 말했던 캘린더 확장함수 빼놨는데 효빈언니 확인 한번만 해줘요!!

boiledEgg-s

너무 고생하셨습니당~ 데이트피커 수정된거 편안하네요!!

boiledEgg-s · 2024-09-04T06:08:49Z

core/src/main/java/com/terning/core/designsystem/component/button/FilteringButton.kt

@@ -87,7 +82,7 @@ fun FilteringButton(
                color = borderColor
            ),
            shape = RoundedCornerShape(cornerRadius),
-            onClick = { onButtonClick() }
+            onClick = onButtonClick


boiledEgg-s · 2024-09-04T06:24:43Z

feature/src/main/java/com/terning/feature/filtering/filteringone/FilteringOneViewModel.kt

+    fun updateButtonValidation() {
+        _state.value = _state.value.copy(isButtonValid = true)
+    }
+
+    fun updateGrade(grade: Int) {
+        _state.value = _state.value.copy(grade = grade)
+    }


최근에 든 의문인데, 예를 들어 어떤 버튼을 누르면 두 함수를 모두 호출하는 상황이 존재한다면 이렇게 나눠서 관리하는게 좋을까요 아니면 용도에 맞게 합쳐서 관리하는게 좋을까요..?
뭔가 이런 사소한 부분 때문에 리컴포지션이 한번이라도 더 발생할 수 있다는 생각이 드는데 맞는지는 모르겠네요..ㅎㅎ 어렵다

저도 관련해서 고민중이었는데 추후에 개별적으로 사용될 가능성을 고려해서 나눠놓는게 좋을까 싶다가도 확실하게 항상 같이 호출하는 상황이라면 합쳐서 관리하는게 더 효율적일 것 같기도 하고... 진짜 어렵다

사실 저도 그 고민이 들긴 했습니다..
하나하나 나누기에는 비효율적인가 싶으면서도 따로 관리해주기엔 용이할 것 같아서..
일단은 상황에 맞게 가져가는 게 좋을 것 같은데 위 화면에서는 둘 다 호출하는 경우만 있기 때문에 합쳐놓도록 하겠습니다!

boiledEgg-s · 2024-09-04T06:26:05Z

...rc/main/java/com/terning/feature/filtering/filteringone/navigation/FilteringOneNavigation.kt

            onNextClick = { grade -> navHostController.navigateFilteringTwo(grade) },
            navigateUp = { navHostController.navigateUp() }


Suggested change

onNextClick = { grade -> navHostController.navigateFilteringTwo(grade) },

navigateUp = { navHostController.navigateUp() }

onNextClick = navHostController::navigateFilteringTwo,

navigateUp = navHostController::navigateUp

이 친구들도 요렇게 바꿀 수 있답니다!

boiledEgg-s · 2024-09-04T06:27:09Z

feature/src/main/java/com/terning/feature/filtering/filteringthree/FilteringThreeRoute.kt

+        onYearChosen = { viewModel.updateStartYear(it) },
+        onMonthChosen = { viewModel.updateStartMonth(it) }


Suggested change

onYearChosen = { viewModel.updateStartYear(it) },

onMonthChosen = { viewModel.updateStartMonth(it) }

onYearChosen = viewModel::updateStartYear,

onMonthChosen = viewModel::updateStartMonth

호출하는 함수가 하나이고, 인자의 순서가 맞다면 얘네도 "::"로 호출이 가능합니닷

boiledEgg-s · 2024-09-04T06:46:27Z

feature/src/main/java/com/terning/feature/filtering/filteringtwo/FilteringTwoViewModel.kt

+    fun updateButtonValidation() {
+        _state.value = _state.value.copy(isButtonValid = true)
+    }


상태를 업데이트를 할 때 MutableStateFlow.update()를 사용하면 멀티 스레드 환경에서 여러 개의 스레드가 동시에 같은 변수를 초기화하는 것을 막아주는 원자성을 보장한대요!
만약 viewModelScope와 같은 코루틴들이 얽힌 경우엔 update를 사용하는 것이 안전할 것 같습니다~

boiledEgg-s · 2024-09-04T06:47:24Z

feature/src/main/java/com/terning/feature/filtering/filteringthree/component/YearMonthPicker.kt

데이트피커 까리해졌네요ㄷㄷ

Hyobeen-Park

데이트 피커 수정한거 진짜 짱이에요!! 확장함수도 확인했습니다요~! 수고하셨습니다😊😊

Hyobeen-Park · 2024-09-06T09:25:12Z

core/src/main/java/com/terning/core/extension/CalendarExt.kt

확인했습니당!! 최고최고💚

Hyobeen-Park · 2024-09-06T09:25:52Z

core/src/main/java/com/terning/core/util/CalendarDefaults.kt

상수로 저장한거 좋네욤!!

Hyobeen-Park · 2024-09-06T09:31:12Z

feature/src/main/java/com/terning/feature/filtering/filteringone/FilteringOneViewModel.kt

+    fun updateButtonValidation() {
+        _state.value = _state.value.copy(isButtonValid = true)
+    }
+
+    fun updateGrade(grade: Int) {
+        _state.value = _state.value.copy(grade = grade)
+    }


저도 관련해서 고민중이었는데 추후에 개별적으로 사용될 가능성을 고려해서 나눠놓는게 좋을까 싶다가도 확실하게 항상 같이 호출하는 상황이라면 합쳐서 관리하는게 더 효율적일 것 같기도 하고... 진짜 어렵다

into ui/#216-on-boarding

leeeyubin added 16 commits September 4, 2024 03:46

[UI/#216] 필터링 시작 화면 수정

3a9f663

[UI/#216] 홈 시작 화면 수정

b65f773

[UI/#216] 필터링 버튼 UI 수정

34542d3

[UI/#216] 필터링 버튼 UI 수정

6211928

[UI/#218] YearMonthPicker 구현

df37526

[MOVE/#218] 파일 위치 이동

90f1c9e

[DEL/#218] 안 쓰는 라이브러리 삭제

8b65715

[FEAT/#218] 데이트피커 수정

9b9c7d2

[FEAT/#218] 캘린더 확장함수 구현

0a643bf

[FEAT/#218] 캘린더 확장함수 구현

a3c673a

[FEAT/#218] 로직 수정

de56489

[FEAT/#218] 로직 수정

aa2531f

[FEAT/#218] 캘린더 오류 수정

b7d263c

[FEAT/#218] 필터링 버튼 로직 수정

990a776

[FEAT/#218] 필터링 원 로직 수정

93ca5c6

[FEAT/#218] 필터링 투 로직 수정

016fc7c

leeeyubin added UI 💐 UI 작업 유빈💙 유빈 labels Sep 3, 2024

leeeyubin added this to the 2차 스프린트 작업 milestone Sep 3, 2024

leeeyubin requested review from arinming, Hyobeen-Park and boiledEgg-s September 3, 2024 23:13

leeeyubin self-assigned this Sep 3, 2024

boiledEgg-s approved these changes Sep 4, 2024

View reviewed changes

leeeyubin added 2 commits September 5, 2024 04:00

[FEAT/#218] resolving conflict

48ee63a

[FEAT/#218] 패딩값 수정

e410bb5

Hyobeen-Park approved these changes Sep 6, 2024

View reviewed changes

leeeyubin added 3 commits September 7, 2024 10:15

[FEAT/#218] 함수 합치기

278d274

[FEAT/#218] 코드 리뷰 반영

fc8c1f3

Merge branch 'develop' of https://github.com/teamterning/Terning-Android

f5d1c60

into ui/#216-on-boarding

leeeyubin merged commit b862c0e into develop Sep 7, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[UI/#218] 온보딩 / UI 수정 #219

[UI/#218] 온보딩 / UI 수정 #219

leeeyubin commented Sep 3, 2024

boiledEgg-s left a comment

boiledEgg-s Sep 4, 2024

boiledEgg-s Sep 4, 2024

Hyobeen-Park Sep 6, 2024

leeeyubin Sep 7, 2024

boiledEgg-s Sep 4, 2024

boiledEgg-s Sep 4, 2024

boiledEgg-s Sep 4, 2024

boiledEgg-s Sep 4, 2024

Hyobeen-Park left a comment

Hyobeen-Park Sep 6, 2024

Hyobeen-Park Sep 6, 2024

Hyobeen-Park Sep 6, 2024

		onNextClick = { grade -> navHostController.navigateFilteringTwo(grade) },
		navigateUp = { navHostController.navigateUp() }

		onYearChosen = { viewModel.updateStartYear(it) },
		onMonthChosen = { viewModel.updateStartMonth(it) }

[UI/#218] 온보딩 / UI 수정 #219

[UI/#218] 온보딩 / UI 수정 #219

Conversation

leeeyubin commented Sep 3, 2024

⛳️ Work Description

📸 Screenshot

📢 To Reviewers

boiledEgg-s 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

Hyobeen-Park 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