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
Feature/jiyun 3 #149
base: main
Are you sure you want to change the base?
Feature/jiyun 3 #149
Conversation
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.
과제 끝내고 다시 올리세요. 기한 제한 없으니 천천히 하세요.
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.
과제완수해주세요 미완으로 보여집니다.
var transitionDuration: Double = 0.8 | ||
var transition: CardTransitionType = .presentation | ||
let shrinkDuration: Double = 0.2 | ||
var originFrame = CGRect.zero |
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인데 여긴 왜 var 인가요?
} | ||
|
||
func setUpUI() { | ||
todayLabel = { |
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.
초기화 코드를 여기에 넣은 이유와 이렇게 한 이유에 대해 설명해주세요.
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.
초기화, layout구성 등 ui를 구성하는 요소에 관한 것들을 모두 한 함수에 넣어서 관리하고 싶었습니다!
label.textColor = .black | ||
return label | ||
}() | ||
dateLabel = { |
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.
공백을 안넣는 것은 습관인건가요?
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.
저는 모두 UI에 관한 구성요소들이기 때문에 굳이 공백으로 나눌 필요가 없다고 생각했습니다.
label.textColor = .gray | ||
return label | ||
}() | ||
profileImageButton = { |
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.
여기도
button.setImage(image, for: .normal) | ||
return button | ||
}() | ||
header = { |
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.
여기도
$0.height.equalTo(800) | ||
$0.width.equalTo(UIScreen.main.bounds.width * 0.9) | ||
} | ||
todayLabel.snp.makeConstraints { |
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.
여기도
$0.bottom.equalTo(todayLabel.snp.top).offset(-10) | ||
$0.leading.equalTo(todayLabel) | ||
} | ||
profileImageButton.snp.makeConstraints { |
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.
여기도
$0.bottom.equalTo(cardTableView.snp.top).offset(-15) | ||
$0.leading.equalTo(cardTableView) | ||
} | ||
dateLabel.snp.makeConstraints { |
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.
여기도
func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) { | ||
let detailView = DetailViewController() | ||
detailView.modalPresentationStyle = .overFullScreen | ||
detailView.transitioningDelegate = self |
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.
detailView는 미완인건가요?
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.
넵 수정하겠습니다
No description provided.