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

🔀 :: (#1259) 아티스트 설명글 짤림 보완 #1260

Merged

KangTaeHoon merged 5 commits into release/3.0.0 from 1259-ui-fix-artist-detail-description

Aug 31, 2024

Contributor

KangTaeHoon commented Aug 31, 2024

💡 배경 및 개요

아티스트 설명글이 짤림

Resolves: #1259

📃 작업내용

텍스트 간격 피그마와 최대한 일치 (설명 글 하단 간격이 쓸데없이 크게 잡혀있었음)
설명글은 한글 텍스트 라인브레이크 적용

🙋‍♂️ 리뷰노트

UILabel 특성상 다각형 레이아웃을 구성할 수 없어 김치만두, 데스해머쵸로키 같은 레이아웃은 구성할 수가 없음.
'우왁굳' 이름 폰트 크기가 맞지않는 것도 옆에 영문 이름이 따라붙어야 하는(줄바꿈 조건 포함) 공통의 조건을 만족시키기 위함.

✅ PR 체크리스트

이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. XCConfig, 노션, README)
이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "XCConfig 값 추가되었어요")
작업한 코드가 정상적으로 동작하나요?
Merge 대상 브랜치가 올바른가요?
PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

KangTaeHoon added 5 commits

August 30, 2024 22:50


          ✨ :: [#1259] 아티스트 설명글 짤림 보완

7acc533


          🎨 :: 코드 Formatting 적용

6ccfae9


          ✨ :: [#1259] 김치만두 예외처리

6735c12


          🎨 :: 코드 Formatting 적용

3e5003d


          ✨ :: [#1259] 캘리칼리 예외처리

cc94d2b

KangTaeHoon requested review from yongbeomkwak, kimdaehee0824, youn9k and baekteun as code owners

August 31, 2024 03:07

github-actions bot assigned KangTaeHoon

github-actions bot commented Aug 31, 2024 •

edited

Loading

✅ Successful finished SwiftLint

github-actions bot commented Aug 31, 2024

✅ 이슈와 PR의 Labels 동기화를 성공했어요!

github-actions bot added 0️⃣ Priority: Critical 🐞 Bug labels

yongbeomkwak approved these changes

View reviewed changes

KangTaeHoon merged commit 7f5ab7f into release/3.0.0

5 checks passed

KangTaeHoon deleted the 1259-ui-fix-artist-detail-description branch

August 31, 2024 03:58

github-actions bot mentioned this pull request

아티스트 상세 > 설명글 짤림 #1259

Closed

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

Reviewers

yongbeomkwak yongbeomkwak approved these changes

kimdaehee0824 Awaiting requested review from kimdaehee0824 kimdaehee0824 is a code owner

youn9k Awaiting requested review from youn9k youn9k is a code owner

baekteun Awaiting requested review from baekteun baekteun is a code owner

Labels

0️⃣ Priority: Critical 🐞 Bug