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.

Sign up for GitHub

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

Jump to bottom

[Feat] MyPageView UI 구현 #530

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

[Feat] MyPageView UI 구현 #530

wants to merge 10 commits into from

Conversation

JMM00
Copy link
Member

@JMM00 JMM00 commented Apr 21, 2024 •
edited
Loading

📌 해결하려는 이슈

✨핵심적인 변경사항

  • MyPageView UI 구현 및 화면 연결
  • 섹션의 접힘/펼침 상태 UserDefaults에 저장
  • 작성한 / 다운받은 / 좋아요 누른 아이템의 수가 3개 이상인 경우에만 아이템 2개 + 더보기 버튼이 나타나도록 설정했습니다.

🔖 그 외 변경사항

  • ExploreShortcutView 와 MyPageView의 섹션 타이틀에 사용되는 이미지를 inner shadow가 적용된 이미지로 변경
  • ExploreShortcutView 하단 여백 수정
  • CardSection 좌우 여백 수정
  • 툴바 아이템 색상 수정 (opcity(0.64) 적용)
  • ExpandedRankingView -> ListShortcutView 이름 변경
  • ListShortcutCell 좌우 여백 수정 ( 검색 결과 리스트 기준으로 적용)
  • Silver 색상 이름을 Gray로 변경
  • navigationLinkRouter 이용하여 화면 연결

전달 사항

  • 프로필 이미지와 닉네임 상단 부분은 추후 디자인이 확정된 이후 수정할 예정입니다.
  • 폰트 종류가 기존 익스텐션과 상이하여 디자이너 확인 받은 후 반영하겠습니다.
  • 아직 확정된 디자인이 아니므로 변경될 가능성이 높습니다.

🙏 리뷰어에게 바라는 점

  • MyPageView 에서 각 섹션이 현재 스크롤이 가능한 상태입니다. 아이템이 한눈에 확인 가능하니 좌우 스크롤 동작을 빼야할까요?

📱 스크린샷

스크롤 시작 스크롤 끝 섹션 모두 접힘
Simulator Screenshot - iPhone 15 - 2024-04-22 at 03 44 27 Simulator Screenshot - iPhone 15 - 2024-04-22 at 03 44 43 Simulator Screenshot - iPhone 15 - 2024-04-22 at 03 44 34

Preview

  • 섹션 접힘 / 펼침 상태 저장 동작
Simulator.Screen.Recording.-.iPhone.15.-.2024-04-22.at.03.44.14.mp4

JMM00 added 9 commits April 20, 2024 18:41
@JMM00
[Feat] #524 - MyPageView UI 구현
032a5f9 
- 518 머지 후 수정 필요
@JMM00
[Feat] OrderedCell 인덱스 번호 수정
947638a
@JMM00
[Feat] #524 - MyPageViewModel 적용
72a7d9a 
- 섹션 접힘 유무 UserDefaults 저장
- 섹션 최대 노출 셀 수 변경
@JMM00
[Feat] #524 - 섹션 폴드 버튼 터치 영역 수정
e2f470f
@JMM00
[Feat] 디자인 수정사항 반영
be5c3b7 
- ExploreShortcutView 섹션 타이틀 아이콘 이미지로 변경
- ExploreShortcutView 하단 여백 수정
- 섹션 더보기 버튼 색상 수정
- 툴바 아이템 색상 수정
@JMM00
[Feat] ExploreCell에 NavigationLink로 버튼 동작 추가
2569a9d 
- 화면은 추후 연결
@JMM00
[Feat] ExpandedRankingView 좌우여백 수정
c595312
@JMM00
[Feat] navigationLinkRouter 이용하여 화면 연결
2f7dbb6 
- Silver -> Gray로 색상명 변경
@JMM00
[Style] ExpandedRankingView -> ListShortcutView 이름 변경
8edb687
HanGyeongjun
HanGyeongjun requested changes Apr 25, 2024
View reviewed changes
Copy link
Member

@HanGyeongjun HanGyeongjun left a comment

Choose a reason for hiding this comment

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

확인했습니다!
추후 레포랑 연결되야겠지만 뷰모델 코드가 너무 깔끔하네요...👀

회의때 말씀해주신 대로 폰트에 대한 부분 적용해주시면 나머지 코드는 문제 없을 것 같아요!
또 섹션별 스크롤도 각 3개씩으로 늘어나서 스크롤 가능하게 유지하는게 좋아 보여요

폰트 익스텐션 부분 커밋되면 바로 어프루브 하겠습니당

HappyAnding/HappyAnding/Assets.xcassets/Icons/loved.imageset/loved.png Outdated Show resolved Hide resolved
HappyAnding/HappyAnding/Views/MyPageViews/MyPageView.swift Show resolved Hide resolved
teethemoji
teethemoji requested changes Apr 29, 2024
View reviewed changes
Copy link
Collaborator

@teethemoji teethemoji left a comment •
edited
Loading

Choose a reason for hiding this comment

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

빠르게 작업해주셨는데 리뷰가 너무 늦어졌네요🥲수고 많으셨습니다 엘리! 수정 제안한 부분만 한번 확인해주시면 될 것 같습니다

@JMM00
Copy link
Member Author

JMM00 commented Apr 29, 2024

빠르게 작업해주셨는데 리뷰가 너무 늦어졌네요🥲수고 많으셨습니다 엘리! 수정 제안한 부분만 한번 확인해주시면 될 것 같습니다

반영했습니다

@JMM00
Copy link
Member Author

JMM00 commented Apr 29, 2024

확인했습니다! 추후 레포랑 연결되야겠지만 뷰모델 코드가 너무 깔끔하네요...👀

회의때 말씀해주신 대로 폰트에 대한 부분 적용해주시면 나머지 코드는 문제 없을 것 같아요! 또 섹션별 스크롤도 각 3개씩으로 늘어나서 스크롤 가능하게 유지하는게 좋아 보여요

폰트 익스텐션 부분 커밋되면 바로 어프루브 하겠습니당

폰트 부분 아직 정리되지 않은 부분이 많아 추후 작업해야할 것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yju0808 yju0808 Awaiting requested review from yju0808

@HanGyeongjun HanGyeongjun Awaiting requested review from HanGyeongjun

@teethemoji teethemoji Awaiting requested review from teethemoji

Requested changes must be addressed to merge this pull request.

Assignees

@JMM00 JMM00

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feat] MyPageView UI 구현
3 participants
@JMM00 @teethemoji @HanGyeongjun