Skip to content

Conversation

@kim-seonwoo
Copy link
Member

@kim-seonwoo kim-seonwoo commented Apr 8, 2024

👾 작업 내용

  • UIPickerView를 도입 하여 Wheel Picker UI를 구현하였습니다.
    SwiftUI Picker에서는 row height 사이즈를 조정하는 것이 불가능했습니다.

  • 공통으로 사용되는 Picker, PickerCoordinator에 대하여 따로 분리를 하였고,
    AppGoalTimePicker와 GoalTimePickerView에서 사용하였습니다.

  • OnboardingCompleteView를 구현 하였습니다.
    OnboardingSurvey관련 부분과는 연결하지 않았습니다. (후에 메인 앱에서 온보딩 종료 bool값에 따라 루트뷰를 바꿀 예정)

  • StoryView로 NavigationLink로 넘어가도록 하였습니다.
    굳이 루트뷰로 안해도 된다고 판단했습니다.

  • StoryContentView에서 @State var StoryState = 0에 따라 내용이 달라지도록 하였습니다.

  • 이 부분도 루트뷰 구현 시 함께 추가하겠습니다.

     if storyState == 3 {
                // TabBarView로 이동 하는 함수로 구현
            }

🚀 PR Point

  • SwiftUI Button의 터치 영역은 label로 한정됩니다.
      .frame(minWidth: 100, maxWidth: .infinity, minHeight: 44, maxHeight: 44, alignment: .center)

따라서 위와 같이 프레임을 보장해 주었습니다.

📸 스크린샷

구현 내용 스크린샷
Picker
Complete
Story

✅ CheckList

  • 오류 없이 빌드되는지 확인
  • 로그용 print문 제거
  • 불필요한 주석 제거
  • 코드 컨벤션 확인

🔗 Issue

Resolved #10

@kim-seonwoo kim-seonwoo added 😎선우 선우의 issue ✨FEAT 기능 구현 🎨STYLE 커스텀 뷰 짜기 labels Apr 8, 2024
@kim-seonwoo kim-seonwoo added this to the 🔭2차 스프린트🪐 milestone Apr 8, 2024
@kim-seonwoo kim-seonwoo requested a review from Zoe0929 April 8, 2024 07:34
@kim-seonwoo kim-seonwoo self-assigned this Apr 8, 2024
@kim-seonwoo kim-seonwoo linked an issue Apr 8, 2024 that may be closed by this pull request
Copy link
Member

@Zoe0929 Zoe0929 left a comment

Choose a reason for hiding this comment

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

👍 스유 천재만재

@kim-seonwoo kim-seonwoo merged commit 07f8f27 into develop Apr 11, 2024
@Zoe0929 Zoe0929 deleted the feat/#10-PickerView branch June 19, 2024 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨FEAT 기능 구현 🎨STYLE 커스텀 뷰 짜기 😎선우 선우의 issue

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Feat] PickerView 추가

3 participants