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

Feat [#17] 마이페이지 뷰 UI 구현 #18

Merged
merged 12 commits into from
May 6, 2024
Merged

Conversation

kim-seonwoo
Copy link
Member

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

👾 작업 내용

  • MyPageView를 구현하였습니다.
    마이 페이지의 뷰를 통합하고 있는 view 입니다.
  • MyPageButton을 구현하였습니다.
    마이 페이지 뷰에 구현되는 네 가지 버튼을 hstack을 이용하여 구현하였습니다.
  • MyPageViewModel을 구현하였습니다.
enum MyPageButtonType {
    case travel
    case market
    case term
    case info
}

위의 네 가지 버튼의 타입에 따라 함수의 동작을 나눴습니다.

🚀 PR Point

  • MyPageView를 보면 Spacer()를 이용하여 gap을 나눴습니다.
    균일 하지 않은 gap들의 경우 padding을 쓰는 편이 차라리 좋을지 싶습니다.
  • 다른 버튼의 동작은 tap 에 대하여 동작 하지 않도록 하였고,
    이용약관, 개인정보는 노션 링크로 연결하였습니다.

📸 스크린샷

구현 내용 스크린샷
마이페이지

✅ CheckList

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

🔗 Issue

Resolved #17

@kim-seonwoo kim-seonwoo added 😎선우 선우의 issue ✨FEAT 기능 구현 labels Apr 15, 2024
@kim-seonwoo kim-seonwoo added this to the 🔭2차 스프린트🪐 milestone Apr 15, 2024
@kim-seonwoo kim-seonwoo self-assigned this Apr 15, 2024
@kim-seonwoo kim-seonwoo linked an issue Apr 15, 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 b3b0591 into develop May 6, 2024
@Zoe0929 Zoe0929 deleted the feat/#17-MyPage 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 기능 구현 😎선우 선우의 issue
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[Feat] 마이페이지 뷰 UI 구현
2 participants