[Feature/#8] : 약속 확정 페이지 구현 #14

gaeulzzang · 2025-01-04T10:07:43Z

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

merge할 브랜치의 위치를 확인해 주세요.(main❌/develop⭕)
리뷰가 필요한 경우 리뷰어를 지정해 주세요.
리뷰는 PR이 올라오면 최대한 빠르게 진행합니다.
P1 단계의 리뷰는 빠르게 확인 후 반영합니다.
Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

closed [UI] : 약속 확정 페이지 구현 #8

📎 𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

입력 가능한 타임테이블 구현
실시간 현황 타임테이블 구현
시간 입력하지 않았을 때 다이얼로그 구현
약속 확정 페이지 구현

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

시간 입력하지 않았을 때	시간 입력했을 때

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

하.... 화면이 너무 많아서 개어지러워요 하다가 토나올뻔했슈... 🤮

…into feature/#8-appointment-confirm-ui

youjin09222

와 가을아 진짜 어케 구현했니..? 증말 멋있다..
고생 넘 많았어ㅓㅓㅓㅓ 👏👏👏

youjin09222 · 2025-01-05T14:10:46Z

core/src/main/java/com/sopt/core/designsystem/component/button/NoostakButton.kt

+    BaseButton(
+        modifier = modifier.fillMaxWidth(),
+        isEnabled = isEnabled,
+        shape = RoundedCornerShape(8.dp),
+        style = NoostakTheme.typography.t3Bold,
+        paddingVertical = 15.dp,
+        text = text,
+        onButtonClick = { onButtonClick() }
+    )


P3: 오 BaseButton 활용하는거 너무 좋습니다!!

youjin09222 · 2025-01-05T14:12:01Z

core/src/main/java/com/sopt/core/designsystem/component/button/NoostakButton.kt

P3: 이 버튼 각자 다 만든걸로 알고있는데 뭘로 통일할까요??

나중에 다 통일해야될 것 같아유.. 😢

youjin09222 · 2025-01-05T14:14:24Z

core/src/main/java/com/sopt/core/designsystem/component/button/BaseButton.kt

@@ -33,7 +33,8 @@ fun BaseButton(
            modifier = modifier,
            shape = shape,
            colors = ButtonDefaults.buttonColors(
-                containerColor = Color.Gray
+                containerColor = if (isEnabled) NoostakTheme.colors.black else NoostakTheme.colors.gray500,


P3: 여기 컬러 값도 받아와서 사용하는걸로 바꾸면 활용도가 높아질 것 같습니다!

컴포넌트 나중에 역할 분담해서 싹다 통일시켜봅시다 ㅠ

presentation/src/main/res/values/strings.xml

youjin09222 · 2025-01-05T14:19:43Z

presentation/src/main/res/values/dimens.xml

    <dimen name="horizontal_padding">16dp</dimen>
+    <dimen name="vertical_padding">16dp</dimen>
+    <dimen name="default_padding">16dp</dimen>


P3: 오 같은 값이여도, 사용되는 곳에 따라 각각 다르게 만들어줘야하나요??

호오옥시 몰라서 다르게 만들었어유... 불필요할까요? (팔랑귀)

P3 : 다르게 만드는게 좋을 것 같아요 각 컴포넌트에 따라 GUI가 달라지면 결국 따로 쓰게 될테니까요

core/src/main/java/com/sopt/core/designsystem/component/timetable/NoostakEditableTimeTable.kt

…into feature/#8-appointment-confirm-ui # Conflicts: # presentation/src/main/res/values/dimens.xml

twogarlic

왐마야 , , 굉장해 엄청나

Eonji-sw

몹시 피곤하네요 정말 수고많으셨어요!

Eonji-sw · 2025-01-08T10:16:32Z

core/src/main/java/com/sopt/core/designsystem/component/timetable/NoostakEditableTimeTable.kt

+                text = text,
+                onClick = {
+                    if (cellType == CellType.Data) {
+                        val cell = rowIndex to columnIndex


P3 : pair 활용 굿

Eonji-sw · 2025-01-08T10:17:50Z

core/src/main/java/com/sopt/core/designsystem/component/timetable/NoostakEditableTimeTable.kt

+                startTime = "%02d:00".format(hour),
+                endTime = "%02d:00".format(hour + 1)


P3 : string export 해도 될둣?

core/src/main/java/com/sopt/core/designsystem/component/timetable/NoostakTimeTable.kt

Eonji-sw · 2025-01-08T10:21:39Z

core/src/main/java/com/sopt/core/designsystem/component/timetable/NoostakTimeTable.kt

+    LazyVerticalGrid(
+        modifier = modifier
+            .border(
+                width = 1.dp,
+                color = NoostakTheme.colors.gray200,
+                shape = RoundedCornerShape(8.dp)
+            ),
+        columns = GridCells.Fixed(days + 1)
+    ) {
+        items((days + 1) * (timeSlots + 1)) { index ->
+            val (rowIndex, columnIndex) = index / (days + 1) to index % (days + 1)
+
+            val cellType = determineCellType(rowIndex, columnIndex)
+            val backgroundColor = getBackgroundColor(cellType, rowIndex, columnIndex, data)
+            val text = getCellText(cellType, rowIndex, columnIndex, data)


P3 : NoostakEditableTimeTable과 상당히 구조가 비슷한데 흠...~

아냐 분리시킬거야. 하나가 될 수 없어.

Eonji-sw · 2025-01-08T10:23:04Z

core/src/main/java/com/sopt/core/designsystem/component/timetable/NoostakTimeTable.kt

+}
+
+@Composable
+fun NoostakTimeTableBox(


P3 : 나중에 코드 재사용쪽으로 리팩 해보면 좋을 것 같긴한데 복잡하긴 하네용 조금씩 달라서

Eonji-sw · 2025-01-08T10:50:54Z

presentation/src/main/java/com/sopt/presentation/appointment/screen/RecommendationScreen.kt

P3 : 이 화면 진짜 복잡한 거 같아요;

우...우우우우우욱 🤮

presentation/src/main/java/com/sopt/presentation/appointment/screen/RecommendationScreen.kt

Eonji-sw · 2025-01-08T10:55:52Z

presentation/src/main/java/com/sopt/presentation/groupCreate/groupCreateSuccess/regex/Regex.kt

P3 : 근데 regex 패키지 안에 더 넣을 파일이 있나요..?

소문자로 네이밍하면 ktlint가 시비 걸어가지고;;;

Eonji-sw · 2025-01-08T10:57:19Z

presentation/src/main/res/values/dimens.xml

    <dimen name="horizontal_padding">16dp</dimen>
+    <dimen name="vertical_padding">16dp</dimen>
+    <dimen name="default_padding">16dp</dimen>


P3 : 다르게 만드는게 좋을 것 같아요 각 컴포넌트에 따라 GUI가 달라지면 결국 따로 쓰게 될테니까요

gaeulzzang added 8 commits December 31, 2024 21:00

#8 [UI] 약속 홈 페이지 구현

117f78e

Merge branch 'develop' of https://github.com/Noostak/Noostak_Android …

6fa92fe

…into feature/#8-appointment-confirm-ui

#8 [UI] 진행 중 약속과 약속 확정 화면 연결

72b76ae

#8 [REFACTOR] string 추출

5a45d3d

#8 [UI] 약속 확정 페이지 구현

68e6850

#8 [UI] 실시간 현황 타임테이블 구현

a2e0c29

#8 [UI] 시간 선택형 타임테이블 구현

83e9a8c

#8 [REFACTOR] 스트링 추출

76b6237

gaeulzzang added 💟 [UI] UI 작업 🍂 가을 가응가 labels Jan 4, 2025

gaeulzzang self-assigned this Jan 4, 2025

gaeulzzang requested a review from a team as a code owner January 4, 2025 10:07

youjin09222 approved these changes Jan 5, 2025

View reviewed changes

gaeulzzang added 10 commits January 6, 2025 18:27

#8 [REFACTOR] 코드리뷰 반영

49497a2

#8 [ADD] 백스택 정리 코드 추가

304c0c0

#8 [MOD] 그룹 상세 멤버 페이지 UI 수정

6375411

#8 [ADD] 약속 확정 페이지 전체 보기 클릭리스너 추가

91c9ee7

#8 [ADD] 타임테이블 예시 추가

5a711b0

#8 [ADD] 시간 선택 뷰 백스택 로직 추가

7c2409c

#8 [MOD] 전체보기, 뒤로가기 로직 수정

03dcfbb

Merge branch 'develop' of https://github.com/Noostak/Noostak_Android …

b9d33bd

…into feature/#8-appointment-confirm-ui # Conflicts: # presentation/src/main/res/values/dimens.xml

#8 [REFACTOR] 하단 버튼 컴포넌트 통일

bb816b1

#8 [RENAME] rename regex, image launcher file

60a1f17

twogarlic approved these changes Jan 7, 2025

View reviewed changes

Eonji-sw reviewed Jan 8, 2025

View reviewed changes

gaeulzzang added 2 commits January 9, 2025 14:20

#8 [REMOVE] LazyVerticalGrid border 제거

b8d30d0

#8 [REMOVE] 불필요한 임포트 제거

726c1b4

gaeulzzang merged commit 58a0470 into develop Jan 9, 2025
1 check passed

Eonji-sw deleted the feature/#8-appointment-confirm-ui branch January 9, 2025 06:29

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feature/#8] : 약속 확정 페이지 구현 #14

[Feature/#8] : 약속 확정 페이지 구현 #14

gaeulzzang commented Jan 4, 2025 •

edited

Loading

시간 입력하지 않았을 때

시간 입력했을 때

youjin09222 left a comment •

edited

Loading

youjin09222 Jan 5, 2025

youjin09222 Jan 5, 2025

gaeulzzang Jan 6, 2025

youjin09222 Jan 5, 2025

gaeulzzang Jan 6, 2025

youjin09222 Jan 5, 2025

gaeulzzang Jan 6, 2025

Eonji-sw Jan 8, 2025

twogarlic left a comment

Eonji-sw left a comment

Eonji-sw Jan 8, 2025

Eonji-sw Jan 8, 2025

Eonji-sw Jan 8, 2025

gaeulzzang Jan 8, 2025

Eonji-sw Jan 8, 2025

Eonji-sw Jan 8, 2025

gaeulzzang Jan 8, 2025

Eonji-sw Jan 8, 2025

gaeulzzang Jan 8, 2025

Eonji-sw Jan 8, 2025

		startTime = "%02d:00".format(hour),
		endTime = "%02d:00".format(hour + 1)

[Feature/#8] : 약속 확정 페이지 구현 #14

[Feature/#8] : 약속 확정 페이지 구현 #14

Conversation

gaeulzzang commented Jan 4, 2025 • edited Loading

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

📌 𝗜𝘀𝘀𝘂𝗲𝘀

📎 𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

시간 입력하지 않았을 때

시간 입력했을 때

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

youjin09222 left a comment • 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

twogarlic left a comment

Choose a reason for hiding this comment

Eonji-sw 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

gaeulzzang commented Jan 4, 2025 •

edited

Loading

youjin09222 left a comment •

edited

Loading