[Design] 마이페이지 UI 구현 by sedah2102 · Pull Request #40 · Honzapda/Honzapda_iOS

sedah2102 · 2024-05-12T13:07:30Z

🔥 PR 타입 (하나 이상 선택)

기능 추가
버그 수정
의존성, 환경 변수, 빌드 관련 코드 업데이트
기타 사소한 수정

🔗 연결된 이슈

Connected: [Design] 마이페이지 UI 구현 #38
이슈 해결 여부

📄 작업 내용

마이페이지 UI 구현(도움말, 로그아웃, 내가 찜한 카페 UI)

구현 내용	15 pro
마이페이지 UI

👀 기타 더 이야기해볼 점

지난 PR 내용과 동일하고 아래 링크 확인해주세요
[Design] 마이페이지 UI 구현 #16

0Hooni · 2024-05-13T06:49:36Z

이거 PR 리뷰중인데 이전 PR 링크주신 부분에 올려주신 화면 영상이 현재 PR의 구현부를 전부 보여주지 못하는 것 같아서, 혹시 영상 새로 찍어서 올려주시거나 그게 어려우시다면 그냥 구현하신거 싹다 한 PR에 다시 적어서 영상 2~3개 정도로 나눠서 PR 올려주시면 리뷰하기 좋을것 같아요!

…to design/#38-design-mypage-ui

AIsiteru98

뷰모델은 안올린건가?
그리고 주요 코드 설명 있으면 더 좋을둣

AIsiteru98 · 2024-05-17T06:38:48Z

Honzapda/Presentation/User/View/HelpMessageView.swift

+            VStack(alignment: .leading, spacing: 18) {
+                VStack(alignment: .leading) { // 도움말 내용
+                    Text("Q. 혼잡도를 어떻게 보나요?")
+                        .foregroundColor(Color.primary06)


이거 .primary06 으로 적어도 됨

수정했습니당

0Hooni

Navigation 관련 코멘트 꼭 확인 하신뒤 개선이 필요해보입니다!

필요하시다면 같이 고민해도 좋습니다👍

0Hooni · 2024-05-17T08:40:58Z

Honzapda/Presentation/User/View/HelpMessageView.swift

+            Spacer()
+        } //: 뒤로가기 버튼
+        ZStack {
+            Color.gray01.frame(maxHeight: .infinity)


혹시 이부분 이해가 안되는데 설명 가능할까요?

수정했습니당

0Hooni · 2024-05-26T07:50:20Z

Honzapda/Presentation/User/View/LogoutAlertView.swift

+
+                    HStack(spacing: 12) { // 버튼
+                        NavigationLink( // 로그아웃
+                            destination: OnboardingView().navigationBarBackButtonHidden()


이렇게 초기 로그인뷰로 가게 되면 그 전까지 쌓은 뷰가 다시 Navigation에 동일하게 쌓이게 되지 않을까 싶어요!

성능적이나 메모리적으로 한번 확인을 거쳐보면 좋을것 같아보이네요☺️

동의합니다!

@youz2me 예전 회의때 잠깐 얘기를 했었는데, 혹시 좋은 방법이 있을까요?

https://stackoverflow.com/questions/57334455/how-can-i-pop-to-the-root-view-using-swiftui

쌓여있던 네비게이션 뷰를 전부 pop해주면 될 것 같아요 ~ 자세한 방법이 위에 나와있어서 가져왔습니다! 한번 참고해보셔용

https://velog.io/@tkddn0518/SwiftUI-NavigationStack-go-to-rootView-NavigationView-활용

이것도 참고해보셔용

youz2me

고생하셨어요 ~~ 충돌난 부분 풀받아서 해결하고 푸시하셔야 할 것 같아서 Request Change로 달아둘게요!

youz2me · 2024-05-27T06:31:15Z

Honzapda/Presentation/User/View/LogoutAlertView.swift

+
+                    HStack(spacing: 12) { // 버튼
+                        NavigationLink( // 로그아웃
+                            destination: OnboardingView().navigationBarBackButtonHidden()


동의합니다!

youz2me · 2024-05-27T06:32:42Z

Honzapda/Presentation/User/View/UserCafeView.swift

+                                    userViewModel.sortColumn == "distance"
+                                    ? Color.white
+                                    : Color.primary05


이부분은 굳이 줄바꿈 안해도 괜찮지 않을까요..? 혹시 Lint쪽에서 오류나는거면 이야기해주세요!

수정했습니당 Lint에서도 오류 안나요!

youz2me · 2024-05-27T06:36:37Z

Honzapda/Presentation/User/View/UserCafeView.swift

+                        Button {
+                            userViewModel.sortColumn = "review"
+                            print(userViewModel.sortColumn)
+                        } label: {
+                            Text("리뷰순")
+                                .lineLimit(1)
+                                .padding(EdgeInsets(top: 8,
+                                                    leading: 12,
+                                                    bottom: 8,
+                                                    trailing: 12)
+                                )
+                                .foregroundColor(
+                                    userViewModel.sortColumn == "review"
+                                    ? Color.white
+                                    : Color.primary05
+                                )
+                                .font(.sCoreDream(.medium, size: 12))
+                                .background(RoundedRectangle(cornerRadius: 20)
+                                    .foregroundColor(
+                                            userViewModel.sortColumn == "review"
+                                            ? Color.primary05
+                                            : Color.white
+                                        )
+                                    .overlay(RoundedRectangle(cornerRadius: 20)
+                                        .stroke(Color.primary05, lineWidth: 1.3)
+                                    )
+                                )


이렇게 중복되는 버튼의 경우에는 ViewModifier 사용해서 코드 간결하게 유지해도 괜찮을 것 같아요 ~~

ViewModifier 사용하는걸로 바꿨습니당

youz2me · 2024-05-27T06:38:10Z

Honzapda/Presentation/User/View/UserCafeView.swift

+                        HStack {
+                            Spacer()
+                            VStack {
+                                Image("icon_my_heart_fill")


Image(.iconMyHeartFill)로 좀 더 간편하게 사용할 수 있어요 ~~ 자체 자동완성도 해주니까 사용해보시면 좋을 것 같습니다

이걸 몰랐네.... 다른 파일들도 가능한건 다 수정했습니다~! 꿀팁 감사해용

youz2me · 2024-05-27T06:38:22Z

Honzapda/Presentation/User/View/UserCafeView.swift

+                            .foregroundColor(Color.gray10)
+                            .font(.sCoreDream(.bold, size: 16))
+
+                        Text(userViewModel.openNow ? "영업 중" : "휴무")


아웅 굿이빈다

UserViewModel추가, Color.삭제, 이미지 수정, 정렬 버튼 ViewModifier 적용

Develop 변경사항 pull 받아서 수정

프로젝트 파일 깨짐 수정, 기존 파일 추가

design: #38 - 마이 UI 구현

984ff3f

sedah2102 linked an issue May 12, 2024 that may be closed by this pull request

[Design] 마이페이지 UI 구현 #38

Open

3 tasks

sedah2102 requested review from 0Hooni, AIsiteru98 and youz2me May 12, 2024 13:07

sedah2102 self-assigned this May 12, 2024

sedah2102 added 💛 주연 작업자 할당 라벨입니다. 🎨 Design UI 구현 시 사용합니다. labels May 12, 2024

sedah2102 added 2 commits May 15, 2024 22:28

Merge branch 'develop' of https://github.com/Honzapda/Honzapda_iOS in…

54f286b

…to design/#38-design-mypage-ui

removed cached

7e5afda

AIsiteru98 approved these changes May 17, 2024

View reviewed changes

0Hooni reviewed May 26, 2024

View reviewed changes

youz2me requested changes May 27, 2024

View reviewed changes

sedah2102 added 4 commits June 2, 2024 21:33

Chore: PR리뷰에 따라 수정

f7c150f

UserViewModel추가, Color.삭제, 이미지 수정, 정렬 버튼 ViewModifier 적용

Add: 마이 Asset 추가

8bfc3ad

chore: #38 - Pull Develop branch

76c9c80

Develop 변경사항 pull 받아서 수정

chore: #38 - 프로젝트 복구 및 기존파일 추가

e9f1a88

프로젝트 파일 깨짐 수정, 기존 파일 추가

Conversation

sedah2102 commented May 12, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔥 PR 타입 (하나 이상 선택)

🔗 연결된 이슈

📄 작업 내용

👀 기타 더 이야기해볼 점

Uh oh!

0Hooni commented May 13, 2024

Uh oh!

AIsiteru98 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

0Hooni left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

youz2me left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

sedah2102 commented May 12, 2024 •

edited

Loading