feat: 홈 탭 협찬·공동구매 카테고리 UI·위치를 탐색 탭과 동일하게 변경 - #41
Conversation
- 언더라인 탭(사료·간식·의류·용품) → pill 칩(식품·의류·장난감·용품) - switchGbHomeCat() 다크 배경 토글로 업데이트 Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- 협찬: 인라인 스타일 칩 → .scroll-cats/.scroll-cat 클래스 적용, filterCat onclick 연결 - 공동구매: switchGbHomeCat 인라인 스타일 방식 → .scroll-cats/.scroll-cat + filterCat으로 통일 - 두 탭 모두 카테고리 바를 스크롤 영역 외부(고정)로 이동, 콘텐츠만 스크롤 Closes #42 Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
bmbyeolhae
changed the title
bmhayoung
left a comment
bmhayoung
left a comment
There was a problem hiding this comment.
코드 리뷰 (#41)
레이아웃 전환(카테고리 바 고정 + 콘텐츠만 스크롤) 구조 자체는 switchTab()이 display:'flex'를 넣어주므로(user.html:1959) 정상 동작합니다. 다만 아래 1번은 머지 전 수정이 필요합니다.
🔴 1. 공동구매 탭 카테고리 필터가 항상 0건이 됩니다
filterCat()은 카드의 textContent에 카테고리 문자열이 있는지로 거릅니다 (user.html:2057).
const text = card.textContent;
card.style.display = text.includes(cat) ? '' : 'none';그런데 renderGbCampaigns()가 만드는 .product-item에는 카테고리 텍스트가 아예 없습니다 — 상품명·브랜드·할인율·가격·기간뿐입니다 (user.html:2166~2179).
즉 식품·의류·장난감·용품 중 무엇을 눌러도 목록이 통째로 사라집니다. 이전에는 switchGbHomeCat()이 토스트만 띄우고 목록은 건드리지 않았으니 명백한 회귀이고, 본문 테스트 항목 5번("해당 카테고리 상품만 필터링되는지")이 실패합니다. 협찬 탭은 c.cat chip을 렌더하기 때문에 정상 동작합니다.
→ GB 카드에도 카테고리 chip을 렌더하거나, filterCat()이 data-cat 속성을 보도록 바꾸는 방향을 제안드립니다.
→ 부수적으로 콜라보 배너는 .product-item이 아니라 필터링해도 계속 남아서, 목록이 빈 채 배너만 뜨는 모양이 됩니다.
🟠 2. "탐색 탭과 동일"이라는 목표와 실제가 어긋납니다
탐색 탭의 공동구매 칩은 전체·사료·간식·의류·용품·뷰티(user.html:374~380)인데, 이 PR의 홈 공동구매는 전체·식품·의류·장난감·용품(= 협찬 세트)입니다. 이슈 #40은 후자를 지시하고 있어 두 이슈가 서로 어긋난 상태입니다. 어느 쪽이 맞는지 기획 확정 부탁드립니다.
🟠 3. switchGbHomeCat()은 죽은 코드인데 수정되었습니다
새 마크업은 전부 filterCat()을 호출하므로 이 함수는 호출처가 없고, 내부 셀렉터 [onclick*="switchGbHomeCat"]도 아무것도 잡지 못합니다. 스타일 수정이 아니라 함수 통째로 삭제가 맞습니다.
🟡 4. 이슈 연결 번호가 잘못됐습니다
Closes #42 — #42는 존재하지 않는 번호입니다(현재 최대 #41). Closes #40, #41로 수정해야 머지 시 자동 클로즈됩니다.
겸사겸사 이슈 #41 본문이 기획 의도 없이 PR 템플릿 그대로 들어가 있어, 이슈만 봐서는 요구사항을 알 수 없습니다.
⚠️ #38과 충돌합니다
#38과 같은 base(user.html 678a801)에서 같은 배너 블록(247~259행) 을 수정하고 있어 충돌이 확정입니다.
이 PR을 먼저 머지하고 #38이 rebase하는 순서를 권장합니다. 반대로 하면 이 PR의 충돌 해결 과정에서 #38이 만든 동적 배너(id 기반)가 하드코딩 배너로 되돌아갈 위험이 큽니다.
- 탐색 탭 공동구매 카테고리를 식품·의류·장난감·용품으로 통일 - filterCat()이 data-cat 속성 우선 확인하도록 수정 (GB 필터 항상 0건 버그 해결) - renderGbCampaigns()에서 product-item에 data-cat 속성 추가 - switchGbHomeCat() 함수 삭제 (호출처 없는 죽은 코드)
2 participants
📝 수정 내역
.scroll-cats/.scroll-cat클래스로 교체,filterCatonclick 연결.scroll-cats/.scroll-cat클래스로 교체,filterCatonclick 연결 (switchGbHomeCat인라인 스타일 방식 제거)filterCat()이data-cat속성 우선 확인하도록 수정 → GB 필터 항상 0건 버그 해결renderGbCampaigns()에서.product-item에data-cat속성 추가switchGbHomeCat()함수 삭제 (호출처 없는 죽은 코드)🖥 화면 변경
⚙️ 데이터/API 변경
🧪 테스트 방법
🔗 Preview URL
https://biteme-wireframes-git-feature-starsun-gb-home-cat-chip-biteme.vercel.app
📎 관련 이슈
Closes #40