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

[UI/YAF-59] 온보딩 알림 시간 설정 UI를 구현 합니다 #18

Merged
merged 18 commits into from
Jan 11, 2025

Conversation

MoonsuKang
Copy link
Member

@MoonsuKang MoonsuKang commented Jan 10, 2025

Related issue 🛠

closed #12

어떤 변경사항이 있었나요?

  • 🐞 BugFix Something isn't working
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 컨벤션에 맞게 작성했습니다. (필수)
  • merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
  • BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

  • 온보딩 Navigation 구조 적립(Conjugation ViewModel SaveStateHandle)
  • 설명, 시간 설정 UI 구현
  • Picker Padding 영역 수정
  • Top, Bottom Bar 구현 및 상황에 맞는 분기처리 진행

Uncompleted Tasks 😅

To Reviewers 📢

  1. 네비게이션 구조를 잡느라 코드수가 조금 많을 수 있습니다.
    AppState 부분은 문제가 없다면 추후 feature:navigator로 옮길 예정입니다.
    혹시 책임분리 관련 문제가 있으면 코멘트 달아주세요.
  2. 테마가 어두워서 화면 전환 시 Activity의 색상이 노출되는 이슈가 있었습니다.
    그래서 찾아보니 Theme에
window.setBackgroundDrawable(ColorDrawable(colors.gray_900.toArgb()))

요걸 넣어주면 된다는 것을 알게되었습니다.

  1. SavedStateHandle을 활용해서 Onboarding 화면에서의 Step관리를 구현해보았습니다.
    최근에 본 기업 과제 테스트에도 사용한 방식 이었어서 적용해봤는데 한 번 보시고 개선할 점을 찾아주시면 감사하겠습니다~

@MoonsuKang MoonsuKang added 📱 UI 사용자 인터페이스 관련 작업 👊 문수문수문수문수문수 labels Jan 10, 2025
@MoonsuKang MoonsuKang requested a review from DongChyeon January 10, 2025 16:59
@MoonsuKang MoonsuKang self-assigned this Jan 10, 2025
Copy link
Member

@DongChyeon DongChyeon left a comment

Choose a reason for hiding this comment

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

LGTM

@MoonsuKang MoonsuKang merged commit 02f4eea into develop Jan 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👊 문수문수문수문수문수 📱 UI 사용자 인터페이스 관련 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] 온보딩 알림 시간 설정 UI를 구현 합니다.
2 participants