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

[S4] 딤 모달 내 스와이퍼 기능 구현 완료 #197

Merged
merged 5 commits into from
Jan 2, 2025
Merged

Conversation

s0zzang
Copy link
Contributor

@s0zzang s0zzang commented Jan 1, 2025

#️⃣ 연관된 이슈

#191


📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 딤 모달 내 스와이퍼 기능 구현
  • 딤 모달 스타일 변경
  • 포트폴리오 딤 모달 내 노출되는 데이터 변경 : 설명 -> 메뉴명
  • 모달 닫기 버튼 z-index 변경

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

Jan-01-2025 16-22-25
⬆️ 최대 3개만 담긴 스와이퍼 예시

  • 포트폴리오 아이템을 클릭하면 딤 모달이 열리고 클릭한 아이템의 앞, 뒤를 포함한 최대 3개의 아이템만 스와이퍼에 담깁니다.
  • 스와이퍼가 3개씩 담기기 때문에 스와이퍼 페이지네이션을 사용할 수 없었습니다.
    따라서 현재 인덱스와 데이터 총 개수를 구하여 페이지네이션을 구현하였습니다.
  • 포트폴리오 목록 데이터를 무한 스크롤 하는 경우는 반영되지 않았습니다. (ㅠ)
  • 난이도가 있었던만큼 상세한 코드 리뷰 부탁드립니다 !

@s0zzang s0zzang merged commit b761f27 into dev Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 기능 추가 UI UI 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants