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

[Chore] #64 - 홈화면, 전체 족보 리스트(BottomSheet) 화면 연결 #92

Merged
merged 13 commits into from
Jul 18, 2024

Conversation

mcrkgus
Copy link
Member

@mcrkgus mcrkgus commented Jul 17, 2024

🔥 Pull requests

👷 작업한 내용

  • 홈화면 연결
  • 홈화면과 전체 족보 리스트를 한 화면에 노출
  • 전체 족보 리스트 BottomSheet 핸들러 동작

🚨 참고 사항

  • Button Action은 api 연동 이후

📸 스크린샷

구현 내용 SE 15 pro
GIF Simulator Screen Recording - iPhone SE (3rd generation) - 2024-07-18 at 06 07 43 Simulator Screen Recording - iPhone 15 Pro - 2024-07-18 at 06 35 39

✅ Check List

  • Merge 대상 브랜치가 올바른가?
  • 최종 코드가 에러 없이 잘 동작하는가?
  • 전체 변경사항이 500줄을 넘지 않는가?

📟 관련 이슈

@mcrkgus mcrkgus added 🔧 Chore Feat 이외에 코드 수정, 내부 파일 수정, 애매한 것들이나 잡일 💞 가현 가현 공주 작업 labels Jul 17, 2024
@mcrkgus mcrkgus self-assigned this Jul 17, 2024
Copy link
Contributor

@shimseohyun shimseohyun left a comment

Choose a reason for hiding this comment

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

너무 고생하였소
해야할 일이 점점 줄어들어 기쁨을 느끼오

Comment on lines +29 to +31
super.init(frame: frame)

}
Copy link
Contributor

Choose a reason for hiding this comment

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

p3 공백이 있다오

Comment on lines +56 to +58
thumbnailImageView.do {
$0.layer.cornerRadius = 8
$0.backgroundColor = .gray
Copy link
Contributor

Choose a reason for hiding this comment

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

p3: 메이크 라운드 함수라는 것을 써보는건 어떠하오?

Comment on lines +138 to +142
menutag.text = "#" + model.menu
hankkiTitle.text = model.hankkiTitle
priceLabel.text = String(model.price) + "원"
likeLabel.text = String(model.liked)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

소인의 부분에서도 이 부분이 쓰였는데, 이 부분을 완전 까먹었었던 것 같소.
이제야 기억을 되 찾았으니 소인도 이부분을 고치겠소

Comment on lines 207 to 209
func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, sizeForItemAt indexPath: IndexPath) -> CGSize {
return CGSize(width: 375, height: 104)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

p3 소인은 잘 모른다만, 이런다면 기기대응이 되는 것이오?

Copy link
Member Author

Choose a reason for hiding this comment

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

미안하오 안일했오

Copy link
Member

@EunsuSeo01 EunsuSeo01 left a comment

Choose a reason for hiding this comment

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

내 기분이 아주 좋소이다.

Comment on lines +10 to +11
/// HomeViewController에서 extension을 통해 Filtering Button의 속성을 변경하는 함수들을 분리
extension HomeViewController {
Copy link
Member

Choose a reason for hiding this comment

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

어여쁘오.

@mcrkgus mcrkgus merged commit 213f272 into develop Jul 18, 2024
@EunsuSeo01 EunsuSeo01 deleted the chore/#64 branch August 13, 2024 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💞 가현 가현 공주 작업 🔧 Chore Feat 이외에 코드 수정, 내부 파일 수정, 애매한 것들이나 잡일
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Chore] 홈화면, 전체 족보 리스트(BottomSheet) 화면 연결
3 participants