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

Feat: 마이리스트, 콜라보리스트 페이지 전체 리스트 조회 무한스크롤 구현 #24

Merged

Conversation

ParkSohyunee
Copy link
Contributor

개요

  • 마이리스트, 콜라보리스트 페이지 전체 리스트 무한스크롤을 구현했습니다.

작업 사항

  • react-query를 사용한 리스트 목록조회 무한스크롤 기능
  • scroll 감지 observer api 커스텀 훅 생성 (useIntersectionObserver)
  • 페이지 관련 컴포넌트 스타일 색상 공용 컬러 변수로 변경
  • 기타 일부 스타일 수정

참고 사항 (optional)

  • 리스트 페이지 추가 작업 예정 목록입니다. 참고 부탁드립니다.
    • 팔로우/팔로잉 기능
    • UI 최종 점검(스켈레톤 UI 추가, 공용 테마 적용 등)
    • 로그인 여부에 따른 로직 수정

스크린샷

Feb-11-2024 15-41-10


리뷰어에게

  • 현재 한번에 불러오는 리스트 개수(size)를 5개로 설정해 놓았는데, QA 테스트 후 size 변경 가능성도 있습니다.
  • observer api 커스텀 훅 사용하시다가 수정이 필요한 부분 있으시면 말씀 부탁드립니다.

Copy link

vercel bot commented Feb 11, 2024

@ParkSohyunee is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@Nahyun-Kang Nahyun-Kang left a comment

Choose a reason for hiding this comment

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

무한스크롤 못하고 있었는데 소현님 PR 보고 다시 도전해봐야겠어요..!🥹 훅 구현해주셔서 정말 감사합니다!!👍🙇‍♀️ 설 연휴에도 고생하셨습니다! LGTM

Copy link
Contributor

@seoyoung-min seoyoung-min left a comment

Choose a reason for hiding this comment

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

소현님, 무한스크롤 훅 구현해주셔서 감사합니다!
몰랐던 리액트 쿼리 개념들도 얻어가네요 🥰
LGTM👍👍👍

@ParkSohyunee ParkSohyunee merged commit 62b0c01 into 8-Sprinters:dev Feb 11, 2024
1 of 2 checks passed
@ParkSohyunee ParkSohyunee deleted the feature/feed-infinite-scroll branch August 3, 2024 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design UI 수정 Feat 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants