-
Notifications
You must be signed in to change notification settings - Fork 2
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] 보관함 캘린더 구현 #212
[Feat] 보관함 캘린더 구현 #212
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.
고생하셨습니다! 👍👍
return "/calendar/\(ym.year)-\(String(format: "%02d", ym.month))" | ||
case .getDiaryByDate(let ymd): | ||
return "/date/\(ymd.year)-\(String(format: "%02d", ymd.month))-\(String(format: "%02d", ymd.day))" |
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.
해당 포맷의 경우는 따로 유지보수하기 편하게 String Extension이나 ymd의 함수로 만들어 놓으면 좋을 것 같아보여요!
.clipShape(RoundedCorners(radius: 15, corners: [.topLeft, .topRight])) | ||
.shadow(color: .black.opacity(0.15), radius: 12, x: 0, y: -5) | ||
.mask { | ||
// 상단에만 shadow를 주기 위함 | ||
Rectangle().padding(.top, -20) | ||
} |
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.
오... 이런 방법이 있군요
Related Issue
Description
Requirements for Review
라이브러리를 뜯어봐서 디버깅이 필요해보입니다.