Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature/#9] : 약속 생성 UI 구현 #28

Closed
wants to merge 9 commits into from
Closed

Conversation

twogarlic
Copy link
Contributor

@twogarlic twogarlic commented Jan 15, 2025

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

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

📌 𝗜𝘀𝘀𝘂𝗲𝘀

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

  • 코리반영
  • 타임피커

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

Screen_recording_20250115_235159.mp4

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

늦어진 점 정말 매우 죄송합니다 ㅠㅅㅠ
소요시간 입력칸은 아직 안정해져서 정해지면 수정하겟슴당
코리 다 반영하려고 햇는데(캘린더 제외하고,,ㅠ 이 부분 계속 해볼게유) 놓친 부분 있다면 알려주세요 !

@twogarlic twogarlic self-assigned this Jan 15, 2025
@twogarlic twogarlic requested a review from a team as a code owner January 15, 2025 14:50
Copy link
Contributor

@gaeulzzang gaeulzzang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

내비게이션 코드는 확실히 깔끔해졌네요!! 리팩하느라 수고 많으셨습니답 ❤️
코리에 반복되는 말이 많은 것 같은데 정리하자면 가로선은 HorizontalDivider로 바꾸기, 패딩 중복 적용되는 문제 수정하기, 버튼 비활성화 여부 확인해서 적용하기입니다!!
다음 버튼으로 이동했다가 백버튼으로 이전 화면으로 돌아가면 사용자가 입력했던 정보가 남아있는게 좋을 것 같죠?? 그럴 땐 단순히 by remember로 정보를 저장하는 것이 아닌 by rememberSaveable을 사용하면 유저가 입력했던 정보를 저장하고 있을 수 있습니다.. 이건 나중에 QA 때 수정하면 될 문제 같구...
사실 모든 파일이 캘린더 디렉토리 안에 있는데... 원래 캘린더에 있으면 안됩니답...ㅠㅠ 약속 생성 부분이니까 appointmentCreate 디렉토리에서 작업하셔야 합니답.. 디렉토리 옮기는게 많이 힘들겠지만 Calendar에서 appointmentCreate 디렉토리로 꼭 이사가주세요 😢
달력 부분은 현재 최대 8일 설정되는 문제랑 7일 이상 선택 시 버튼이 사라진 상태로 토스트 메시지가 뜨는 문제도 해결해주세요!
타임피커 부분은 시간 선택하고 있을 때 다른 부분 클릭하면 시간이 검은색으로 바뀌는 문제 해결해주세요.. 고르고 있을 땐 계속 파란색이어야 합니답

피커 자체가 난이도 너무 높았을 텐데 fling도 적용하고 무한 스크롤 너무 야무지게 잘해서 깜짝 놀랐어욤 너무 수고 많으셨습니답... 열심히 리팩해서 QA 때 적게 일합시다 흙흙 수고해써 ☁️ 언뉘 🫶
많이 힘들테니까 내 남친 소개할게. 잘생긴 용안 널리 공유합니다.
image

var endDate by remember { mutableStateOf(end) }
var showMessage by remember { mutableStateOf(false) }

val yearMonth = YearMonth.of(year, month)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P3: YearMonth가 ISO8601 시간 형식 바꿔주는 라이브러리군요!! 첨 알았어여... 😢

Comment on lines +166 to +173
BasicTextField(
value = duration,
onValueChange = { duration = it },
modifier = Modifier.width(50.dp),
textStyle = typography.b1SemiBold,
singleLine = true,
keyboardOptions = KeyboardOptions(keyboardType = KeyboardType.Number)
)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1: 여기 indicator 어디갔어요!!??

@twogarlic twogarlic closed this Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] : 약속 생성 UI
2 participants