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: 콜렉션 리스트 무한스크롤 기능 구현 및 일부 로직 수정 #268

Merged
merged 13 commits into from
Oct 29, 2024

Conversation

ParkSohyunee
Copy link
Contributor

@ParkSohyunee ParkSohyunee commented Oct 29, 2024

개요

  • 콜렉션 페이지 리스트 무한스크롤 기능 추가 및 일부 로직 수정, 디자인을 개선했습니다.

작업 사항

  • 콜렉션 폴더 조회 API 수정으로 관련 로직 수정
    • [폴더이름] 필드가 추가됨에 따라 헤더에 현재 폴더이름이 나오도록 구현
    • 폴더 이름 [수정]시에 바텀시트에 기존 폴더이름 보이도록 구현
    • 폴더 이름 수정 후 콜렉션 리스트 최신 데이터 조회 로직 추가
  • 콜렉션 리스트 무한스크롤 기능 구현
  • 디폴트 폴더인 [전체] 폴더 추가
    • 전체 폴더일때는 헤더에 [수정]버튼 옵션이 노출되지 않도록 적용
  • 리스트 디자인 배경 투명도 적용 및 기타 스타일 수정
  • NoData UI 추가

참고 사항 (optional)

  • 추후 해당 코드 리팩토링 예정입니다.

스크린샷

전체 폴더 추가

image

콜렉션 리스트

  • 배경이 밝은 리스트의 경우 글씨가 잘 안보여서 background에 그라디언트 & 투명도를 넣었습니다. (기존보다 아주살짝 어두워짐)
image

리뷰어에게

  • 감사합니다. ✨

Copy link

vercel bot commented Oct 29, 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.

콜렉션 언제봐도 너무 깔끔하고 좋습니다..🥹 저도 무한스크롤 구현할 때 교본삼아 보며 구현하겠습니다..!

@@ -35,7 +40,7 @@ export default function HeaderContainer({ handleSetOnBottomSheet, handleSetOnDel

return (
<>
<Header title="콜렉션" left="back" right={<RightButton />} leftClick={() => history.back()} />
<Header title="콜렉션" left="back" right={!isHideOption && <RightButton />} leftClick={() => history.back()} />
Copy link
Contributor

Choose a reason for hiding this comment

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

@ParkSohyunee 콜렉션 '전체'가 생긴 후 기획안을 세워두지 않았는데, 세심한 부분까지 챙겨주셔서 진심으로 감사드립니다🙇‍♀️

@ParkSohyunee ParkSohyunee merged commit 7d0789a into 8-Sprinters:dev Oct 29, 2024
1 of 2 checks passed
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