[FEAT] 2.0 설정 UI (#154) #161
Conversation
|
|
||
| let glassmorphismView = GlassmorphismView(attributes.glassmorphismType) | ||
| let glassBorderView = GlassmorphismView(attributes.glassmorphismType) | ||
| glassBorderView.tag = 250603 |
There was a problem hiding this comment.
🐿️ 🐿️ 🐿️ tag를 하드코딩하는 대신 안전하게 상수로 선언하면 어떨까요?
UIView+ 안에 private static let으로 선언할 수 있을 것 같습니다!
| } | ||
| } | ||
|
|
||
| // 개별 업데이트는 데이터가 이미 로드된 후에만 |
| override func setLayout() { | ||
| glassmorphismView.snp.makeConstraints { |
There was a problem hiding this comment.
🐿️ super.setLayout()은 의도적으로 생략하신걸까요?
| override func setStyle() { | ||
| self.do { |
There was a problem hiding this comment.
🐿️ 여기도 super.setStyle()은 의도적으로 생략하신걸까요?
| titleLabel.setLabel(text: data.title, style: .s1) | ||
| func configure(with data: String) { | ||
| titleLabel.setLabel(text: data, style: .t4R) | ||
| if data == "서비스 탈퇴" { |
There was a problem hiding this comment.
🐿️🐿️ 나중에 텍스트가 바뀔 때를 대비해서 좀 더 안전하게
if data == SettingType.Account.withdrawal.title 이렇게 하면 어떨까요?
| $0.setLabel(text: "최신버전", | ||
| style: .s2, | ||
| color: .gray400, | ||
| style: .b1R, | ||
| color: .gray500, | ||
| alignment: .right) | ||
| } | ||
|
|
||
| updateLabel.do { | ||
| $0.isHidden = true | ||
| $0.setLabel(text: "업데이트하러 가기", | ||
| style: .s2, | ||
| color: .gray400, | ||
| style: .b1R, | ||
| color: .labelAction, | ||
| alignment: .right) | ||
| } |
There was a problem hiding this comment.
🐿️ text는 StringLiterals로 설정해도 좋을 것 같습니다!
| extension VerifiedAreasCollectionViewCell { | ||
|
|
||
| func bindData(_ title: String, _ indexRow: Int) { | ||
| label.setLabel(text: title, style: .b1R) | ||
| self.setNeedsLayout() | ||
| self.layoutIfNeeded() | ||
| DispatchQueue.main.async { | ||
| self.glassmorphismView.refreshBlurEffect() | ||
| } | ||
| } | ||
|
|
||
| } | ||
|
|
||
| extension VerifiedAreasCollectionViewCell { | ||
|
|
||
| func setAddButton() { | ||
| [glassmorphismView, deleteButton].forEach { | ||
| $0.isHidden = true | ||
| } | ||
| label.setLabel(text: StringLiterals.Profile.addVerifiedArea, | ||
| style: .b1R, | ||
| color: .labelAction, | ||
| alignment: .center) | ||
| label.sizeToFit() | ||
|
|
There was a problem hiding this comment.
🐿️ label.sizeToFit()을 bindData()에서는 안 해줘도 되나요?!
There was a problem hiding this comment.
alignment: .center로 지정해줘서 레이블 크기를 다시 계산해야 해 label.sizeToFit이 필요한 것 같은데, 미리 오토레이아웃으로 크기를 딱 맞게 지정해서 alignment 속성 지정이 필요 없는 것 같아요 !
setAddButton이 alignment 설정과 label.sizeToFit을 지웠습니다
|
|
||
| // MARK: - Properties | ||
|
|
||
| private var flowType: OnboardingFlowType |
🐿️ Pull Requests
🪵 작업 브랜치
🥔 작업 내용
2.0 설정 UI 작업 및 작은 작업들 진행했습니다.
[ 2.0 설정 UI 작업 ]
[ 그 외 작업 ]
🚨 참고 사항
📸 스크린샷
💥 To be sure
🌰 Resolve issue