Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Fix] 관심장르 empty인 경우 separator 위치 조정 #140

Merged
merged 2 commits into from
Jul 15, 2023

Conversation

JongHoooon
Copy link
Member

@JongHoooon JongHoooon commented Jul 13, 2023

관심 장르 empty인 경우 label에 " " 넣어줘 라벨 크기 갖도록 해 separator 위치 조정

#139

관심 장르 empty인 경우 label에 " " 넣어줘 라벨 크기 갖도록 해 separator 위치 조정
@JongHoooon JongHoooon added the Fix label Jul 13, 2023
@JongHoooon JongHoooon self-assigned this Jul 13, 2023
@seungwon-yoo
Copy link
Contributor

" "을 넣어서 해결하는 게 방법보다는 UI 자체를 수정하는 방법이 좋을 것 같습니다! 나중에 히스토리를 모르는 누군가가 봤을 땐 왜 이렇게 짰는지 모를 수 있어서요

minHeight = genreLabel height + top inset + bottom inset
@JongHoooon
Copy link
Member Author

JongHoooon commented Jul 14, 2023

수정했습니다!

@JongHoooon JongHoooon merged commit f1d37d8 into develop Jul 15, 2023
1 check passed
@JongHoooon JongHoooon deleted the fix/editProfileSeparator branch July 15, 2023 04:46
JongHoooon added a commit that referenced this pull request Jul 17, 2023
* [Fix] 관심장르 empty인 경우 separator 위치 조정

관심 장르 empty인 경우 label에 " " 넣어줘 라벨 크기 갖도록 해 separator 위치 조정

* [fix] genreView의 최소 높이 constraint 추가

minHeight = genreLabel height + top inset + bottom inset
JongHoooon added a commit that referenced this pull request Jul 17, 2023
* [Fix] 관심장르 empty인 경우 separator 위치 조정

관심 장르 empty인 경우 label에 " " 넣어줘 라벨 크기 갖도록 해 separator 위치 조정

* [fix] genreView의 최소 높이 constraint 추가

minHeight = genreLabel height + top inset + bottom inset
JongHoooon added a commit that referenced this pull request Jul 17, 2023
* [Fix] 관심장르 empty인 경우 separator 위치 조정

관심 장르 empty인 경우 label에 " " 넣어줘 라벨 크기 갖도록 해 separator 위치 조정

* [fix] genreView의 최소 높이 constraint 추가

minHeight = genreLabel height + top inset + bottom inset
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants