-
Notifications
You must be signed in to change notification settings - Fork 5
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] 메뉴탭에 메뉴판 전체보기 추가 #234
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.
고생선배~ 구글링해보니 섹션 헤더 높이 0으로 두는것 정도밖에는 안보이네요 일단~ 수고하셨슴당!
let sectionType = SectionLayout.allCases[section] | ||
let count : Int = sectionType == .menu ? menuData.count : 1 | ||
return count |
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.
p5;
이쁘게 잘쓰셨네요..와우
@@ -99,6 +105,9 @@ extension MenuTabVC { | |||
|
|||
private func registerCell() { | |||
MenuCellCVC.register(target: menuCV) | |||
AllImageCVC.register(target: menuCV) | |||
menuCV.register(ImageHeaderView.self, forSupplementaryViewOfKind: UICollectionView.elementKindSectionHeader, withReuseIdentifier: ImageHeaderView.identifier |
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.
p5;
별건 아니지만 reuseIdentifier 쓸 때 .className으로 넣어줄 수 있겠네요!
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.
역시 영린이 난 영베만 믿고 있었어
🔥 Pull requests
⛳️ 작업한 브랜치
👷 작업한 내용
메뉴탭에서 메뉴판 전체보기 추가
🚨 참고 사항
메뉴판 이미지와 메뉴판 이미지 클릭 시 확대되어 보이는 이미지들은 임시로 넣어뒀습니다. 서버 붙일때 수정하겠습니다~!
그리고..
질문이 있습니다 !
컬렉션뷰 헤더를 지정할때 모든 섹션에 "메뉴 전체보기" 라는 헤더가 생겨서 헤더 사이즈 지정하는 부분에서 menu section부분의 높이를 0으로 지정해서 안보이게 해두었는데 다른 좋은 방법이 없을까요? 도저히 모르겠습니다 ㅎㅎ
📸 스크린샷
용량이슈로 gif가 안올라가네요..
https://user-images.githubusercontent.com/75068759/187742929-7014a662-537c-4910-a1ea-58fc989831c6.MP4
📟 관련 이슈