Skip to content

Conversation

@junye0l
Copy link
Member

@junye0l junye0l commented Oct 17, 2025

📄 PR 내용 요약

  • 더보기 버튼 로직 수정
  • 향 컴포넌트 오류 수정
  • 좋아요 낙관적 업데이트 반영
  • 리뷰 등록,삭제,수정시 스크롤 위치 수정
  • 리뷰 업데이트 시 반영 되게끔 수정

✅ 작업 내용 상세

더보기 버튼

  • 기본 상태 : 총 리뷰가 5개 이하이면 항상 버튼을 숨김, 5개 초과하면 기본으로 5개만 노출하고 버튼 표시

  • 리뷰 추가 : 총 개수가 늘어나면 표시 개수를 다시 5개로 리셋하고 버튼 유지 ( 5개 초과일 때만 )

  • 리뷰 삭제 : 더보기를 펼쳐둔 상태라면 리뷰를 삭제해도 그 상태 유지, 삭제 후 리뷰갯수가 5개 이하면 버튼 숨김

  • 더보기 클릭 : 리뷰를 5개씩 늘려가되 총 리뷰 수를 넘지 않도록 조절


향 컴포넌트

  • 기존 향 이미지 크기를 100x100으로 일괄 통일

  • 기존 태블릿 향 갯수 4개에서 3개로 조정

  • Flavor 컴포넌트 내에 작성된 텍스트 중복 표시를 막기 위해 Props 추가


좋아요 버튼

  • mutation 로직을 낙관적 업데이트 방식으로 수정하여 작성하였습니다.

리뷰 등록,수정,삭제 관련

  • 등록,수정,삭제 시 전체 페이지가 리렌더링 되어 스크롤이 최상단으로 이동하는 것을 로직을 수정하여 스크롤 위치가 유지되는 방식으로 변경하였습니다.

📸 스크린샷 (선택사항)

태블릿 향 컴포넌트 수정 사진

스크린샷 2025-10-17 17 58 12

💬 참고 사항

  • 로컬 환경에서는 문제가 없는 것 같습니다. 추후 배포 후에 확인하면 좋을 것 같습니다.

  • 로컬에서 테스트하는 와중에 낙관적 업데이트으로 수정한 브랜치로 남긴 리뷰들이 해당 PR 브랜치에서 오류가 생겨
    낙관적 업데이트를 도입한 브랜치의 코드를 해당 브랜치로 merge하여 추가 수정하였습니다.

  • page-modal.tsx, confirm-modal 스크롤 유지 로직이 있었지만 페이지가 리렌더링 되면서 스크롤이 0으로 초기화 되는 현상이 있어
    requestAnimationFrame로 감싸 렌더링 완료 후에 스크롤이 복구되도록 코드 한 줄 수정하였습니다.

@junye0l junye0l requested review from huuitae, suuuuya and wlrnjs October 17, 2025 09:02
@junye0l junye0l self-assigned this Oct 17, 2025
@junye0l junye0l added this to WHYNE Oct 17, 2025
@junye0l junye0l added 🎨 Design UI/UX, 디자인 변경 🔨 Refactor 코드 리팩토링 labels Oct 17, 2025
@vercel
Copy link

vercel bot commented Oct 17, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
wine Ready Ready Preview Comment Oct 18, 2025 5:41am

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

@junye0l junye0l moved this to 검토 중 in WHYNE Oct 17, 2025
@junye0l junye0l linked an issue Oct 17, 2025 that may be closed by this pull request
5 tasks
Copy link
Contributor

@suuuuya suuuuya left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!!

@junye0l junye0l merged commit 340397c into develop Oct 18, 2025
3 checks passed
@junye0l junye0l deleted the refactor/detail-page branch October 18, 2025 05:41
@github-project-automation github-project-automation bot moved this from 검토 중 to 완료 in WHYNE Oct 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨 Design UI/UX, 디자인 변경 🔨 Refactor 코드 리팩토링

Projects

Status: 완료

Development

Successfully merging this pull request may close these issues.

[Refactor] 좋아요 낙관적 업데이트 [Refactor] 더보기 버튼 개선

3 participants