[Style/#19] 아티스트 멤버 섹션뷰 / 내 활동 섹션뷰 구현#35
Open
sssthnnhee wants to merge 14 commits into
Open
Conversation
3 tasks
dearosmar
reviewed
May 13, 2026
Collaborator
dearosmar
left a comment
There was a problem hiding this comment.
전체적으로 코드의 순서에 대한 공통적인 고민이 조금 필요해 보여요! 프로퍼티 - 초기화 - 오버라이드 - 추가 함수 이런 식으로 구현해 보시는 건 어떠신가요? 그리고 코드를 공통적인 방식으로 구현해 보시면 익숙해지면서 편해지실 것 같습니다! 새벽까지 너무 고생하셨습니다
sangyup12
approved these changes
May 14, 2026
Collaborator
sangyup12
left a comment
There was a problem hiding this comment.
고생하셨습니다! 컬뷰 잘하시는데 컴포지셔널도 해보심이 어떨지 ㅎㅎ .아쉽게도 더이상 구현할 컬뷰가 없네요. 폴더 구조까지 생각해주시다니 정말 훌륭합니다. setInfoLabel은 말씀해주신대로 setStyle에서 바로 호출해주시면 될 거 같습니다 !
마크 주석 위아래 한칸씩 띄워주는 것만 신경써주시면 좋을 거 같아요 !!
| private let artistLabelKr = UILabel() | ||
|
|
||
| private let artistLabelEn = UILabel() | ||
| //MARK: - Initializer |
Collaborator
There was a problem hiding this comment.
마크 주석은 위에도 한 칸 띄워서 써줍시다 ~
dearosmar
approved these changes
May 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🍈 작업 내용
( 기존에 작업해둔 헤더 컴포넌트와 컬렉션 뷰 사용 )
( 이번에 새로 필요한 컬렉션 뷰 및 컴포넌트 구현 )
🍈 주요 코드 설명
🍉 라벨 내 색상 지정
MyActivitySectionViewprivate func setInfoLabel() { let name = "김렛솝" let text = "\(name)님은 3개의 카드를 가지고 있어요." let coloredText = NSMutableAttributedString(string: text) let range = (text as NSString).range(of: name) coloredText.addAttribute(.foregroundColor, value: UIColor.green02, range: range) myCardInfoLabel.attributedText = coloredText } private func setMyCardInfo(){ setInfoLabel() }setMyCardInfo라는 함수에서 호출하도록 하였는데, 이 부분에 대해 고민중입니다 (PR 맨 아래 참조 ... + 코드리뷰를 기다림)그리고 새로 추가한 폴더와 파일이 여러 개 있어서..
폴더구조 설명 간단하게 드리고 싶습니다
🍉 목표하는 폴더 구조
우선 위와 같은 구조로 파일을 작성해보았고, 제가 희망하는 최종 파일구조는 아래와 같습니다
(*위 빈 파일들은 로컬 테스트용 브랜치에 만들어본것이니 안심하십시오 지금 푸시해두지않앗습니다)
🍉 섹션 뷰 사용 시 유의점
당연히 알고계시겠지만 꼭 제약조건을 설정해주십시오...
🍈 구현화면
🍈 연결된 이슈
🍈 기타 더 이야기해볼 점
MyActivitySectionView파일에서setUI() 안에 둔
myCardInfoView.addSubviews(myCardInfoLabel,myCardChevron)이 코드를
setMyCardInfo에 넣었었는데, addSubviews 는 여태까지 전부 setUI 에 넣어왔다는 것을 깨닫고 다시 setUI로 옮겼습니다그래서 사실상
setMyCardInfo는 함수 하나만 호출하는거라 의미가 없는 것같고..이
setMyCardInfo를 setStyle 에서 부르고있어요그냥 setStyle 내에서 바로
setInfoLabel해주는게 나을까 라는 고민입니다