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

Home UI 구현 #16

Merged
merged 13 commits into from
Jan 29, 2024
Merged

Home UI 구현 #16

merged 13 commits into from
Jan 29, 2024

Conversation

WhiteHyun
Copy link
Member

@WhiteHyun WhiteHyun commented Jan 28, 2024

Screenshots 📸

Home 화면
Screen Recording 2024-01-28 at 4 50 57 PM
en ko jp
IMG_3419 IMG_3421 IMG_3420

Note

서버로부터 내려받는 제품목록은 아직 한글이어서 백엔드로부터 데이터를 받을 때 이를 수정해야합니다.



고민, 과정, 근거 💬

  • 아직 아이콘 디자인이 전부 정리되지 않아 SF Symbol로 전부 적용해두었습니다. 디자인 측에서 아이콘 정리가 되는대로 업데이트할 예정입니다.
  • 아직 해당 PR에서 ProductInfoHeader UI 구현 #9 가 merge되지 않은 상황에서 올리는 거라 임의의 View를 그려두었습니다. #9가 머지되는대로 PromotionTagViewtoStringWithComma()로 변경할 예정입니다.

트러블 슈팅

Localization 기능을 지원하는 것 까지는 좋았으나 부분 문자열의 색상을 변경하는 것은 꽤나 고역이었습니다.
Text 메서드에서 부분 문자만 색상처리하는 modifier는 존재하지 않는 것 같더라고요.
그래서 WWDC 영상 자료좀 찾아보다가 AttributeString(localized:)를 알게 되었습니다.[2]
Localizable에서 작업한 문자열을 적용한 뒤 AttributedSubString에 존재하는 foregroundColor값을 수정하여 해결했습니다.
WWDC 최고.. 🙌



References 📋

  1. How to adjust List row separator insets in SwiftUI
    • List로 제품목록을 보여줄 때 이미지 부분까지 separator가 가지 않는 것을 확인하고, 해결방법을 모색하던중 찾게된 블로그
  2. Unlock the power of grammatical agreement




@WhiteHyun WhiteHyun added 🏠 Home Home View 🔨 Implementation New feature or request labels Jan 28, 2024
@WhiteHyun WhiteHyun added this to the v2.0.0 milestone Jan 28, 2024
@WhiteHyun WhiteHyun requested a review from a team January 28, 2024 08:38
@WhiteHyun WhiteHyun self-assigned this Jan 28, 2024
@WhiteHyun WhiteHyun linked an issue Jan 28, 2024 that may be closed by this pull request
eung7

This comment was marked as resolved.

Copy link
Contributor

@eung7 eung7 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! SwiftUI는 파일이 너무 많이 생겨버려서 고민이었는데,
이렇게 관련된 파일 안에 View 구조체를 많이 만드는 것도 하나의 방법이겠네요!
저도 @WhiteHyun 님 처럼 WWDC나 공식문서를 먼저 참고하는 습관을 들여야겠습니다..! 👍👍

수고하셨습니다. 😎

@eung7 eung7 requested a review from isGeekCode January 28, 2024 22:20
@WhiteHyun WhiteHyun requested a review from eung7 January 29, 2024 01:19
@WhiteHyun WhiteHyun merged commit 7eb3c77 into main Jan 29, 2024
2 checks passed
@WhiteHyun WhiteHyun deleted the feature/home/5 branch January 29, 2024 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏠 Home Home View 🔨 Implementation New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

Home UI를 구성한다.
2 participants