Skip to content
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

CalendarCell 구현 #8

Merged
merged 4 commits into from
Nov 15, 2022
Merged

CalendarCell 구현 #8

merged 4 commits into from
Nov 15, 2022

Conversation

wickedRun
Copy link
Collaborator

@wickedRun wickedRun commented Nov 14, 2022

📕 Issue Number

Close #3

📙 작업 내역

구현 내용 및 작업 했던 내역

  • CircleCheckView 커스텀 뷰 구현

Screenshot 2022-11-14 at 11 36 17 PM

  • CalendarCell 커스텀 셀 구현

Screenshot 2022-11-14 at 11 34 52 PM

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

📋 체크리스트

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?
  • 새로운 테스트와 기존의 테스트가 변경사항에 대해 만족하는가?

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 컬러값의 경우에는 하드 코딩을 했으며, 같이 코드 합친 후에 변경 하면 좋을 것 같아요.

  • 고민한 점

    • 체크마크를 라벨로 할지 이미지 뷰로 할지 고민했는데 이미지 뷰로 했을 때 굵기를 어떻게 처리할지 몰라서 라벨로 구현했습니다.


Copy link
Collaborator

@jinwoong16 jinwoong16 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

늦은 밤까지 고생많이하셨어요~!
오늘 푹 쉬시면 좋겠네요! 꿀잠자세요!

if range ~= number {
self.displayLabel.text = "\(number)"
} else {
self.displayLabel.text = "9+"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

9를 초과하면 +로 표기해주는 센스 굳입니다.

@sprituz
Copy link
Member

sprituz commented Nov 14, 2022

고생하셨읍니다.

@jinwoong16 jinwoong16 merged commit e1c6705 into develop Nov 15, 2022
@wickedRun wickedRun deleted the feature/calendarCell branch November 15, 2022 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants