-
Notifications
You must be signed in to change notification settings - Fork 1
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
[PR] 약속 - 약속 수정/삭제 리팩토링 #46
Conversation
// 약속 수정페이지 설정 | ||
func setupEditPlanPage(_ viewController: MakePlanViewController) { | ||
viewController.titleLabel.text = "약속 수정하기" | ||
|
||
viewController.planNameTextField.text = planTitle.text | ||
viewController.planStartDateField.text = sendDate[0].replacingOccurrences(of: "-", with: ". ") | ||
viewController.planStartDateField.textColor = UIColor.textDisabled | ||
viewController.calendarButton.isEnabled = false | ||
|
||
viewController.cautionLabel.text = "약속 기간은 수정할 수 없습니다." | ||
viewController.cautionLabel.textColor = UIColor.textDisabled | ||
viewController.cautionImage.isHidden = true | ||
|
||
viewController.planStartDateField.isUserInteractionEnabled = false | ||
viewController.makeButton.setTitle("수정", for: .normal) | ||
viewController.planId = planId | ||
} | ||
|
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.
오오 이렇게 뺀 건 좋은 것 같아여!
그런데 어쩌다가 이렇게 nextVC에 설정해야할 내용이 많아진건가요..??
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.
오우 이건 원래 있던 설정들인데 따로 함수화하면 깔끔해 보일것같아서 함수화를 해놓은거랍니다~~
PR Type ⚙️
What is this PR? 📝
Related Issue Number
Changes
Other information 🔥