-
Notifications
You must be signed in to change notification settings - Fork 0
[Feat] 온보딩 화면 구현 #137
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
[Feat] 온보딩 화면 구현 #137
Conversation
…l cornerRadius 수정
JinUng41
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다.
간단한 코멘트만 남겨두었으니, 확인 부탁드려요!
| let navigationController = UINavigationController(rootViewController: LCKYearViewController(type: .flow)).then { | ||
| $0.navigationBar.isHidden = true | ||
| $0.modalPresentationStyle = .fullScreen | ||
| } | ||
|
|
||
| self.present(navigationController, animated: true) | ||
| })) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
메모리 누수의 여지가 보이는데, weak 또는 unowned 키워드를 사용해야 할 것 같네요!
| guard let url = URL( | ||
| string: sender == rootView.personalInfoAgreementItemView.infoButton ? "https://joyous-ghost-8c7.notion.site/c6e26919055a4ff98fd73a8f9b29cb36?pvs=4" : "https://joyous-ghost-8c7.notion.site/fff08b005ea18052ae0bf9d056c2e830?pvs=4" | ||
| ) else { | ||
| return | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
링크를 상수화 하는 것도 좋아보이네요.
enum Constant {
static let tempURL1 = ""
static let tempURL2 = ""
}| for cell in collectionView.visibleCells { | ||
| guard let cell = cell as? LCKTeamCollectionViewCell else { return } | ||
|
|
||
| cell.layer.borderColor = UIColor.gray300.cgColor | ||
| cell.teamLabel.textColor = .gray700 | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
타입 캐스팅에 따른 조건문에서 만약 실패하여, return을 만났을 경우를 생각해 보면 좋을 것 같습니다.
왜냐면 return을 만났을 때, 실행하고 있는 메서드를 종료시키고 제어권을 반환하기 때문이죠.
반복문을 스킵하고 싶다면, continue가 맞을 것 같네요!
👻 PULL REQUEST
📄 작업 내용
💻 주요 코드 설명
현재 날짜 기준 LCK 시작 연도 계산
Calendar.current.component(.year, from: Date())를 이용해 현재 날짜 기준 연도를 계산하고 LCK 시작 연도인 2012년부터 현재 연도까지 CollectionViewCell에 담기도록 구현했어요.👀 기타 더 이야기해볼 점
🔗 연결된 이슈