Skip to content

Conversation

@0Hooni
Copy link
Member

@0Hooni 0Hooni commented Apr 4, 2025

📌 이슈

✅ 작업 사항

  • ImageBannerSectionCell.swift파일의 inject() 메서드 수정
    • CollectionView에 사진이 한장일 때 스크롤을 방지하도록 수정
  • ImageBannerSectionCell.swift파일의 collectionView(), didEndDisplaying 델리게이트 메서드 수정
    • 사진은 한장인 상태에서 이미지가 사라질 때 index out of range가 발생하는 문제를 수정

🚀 테스트 방식

단일 사진 스크롤 방지

0Hooni added 5 commits April 4, 2025 10:51
- 셀에서 벗어날 때 해당 델리게이트 메서드가 동작
- 하지만 셀에 이미지가 한장일 때 해당 메서드가 out of index를 유발함
- 해당 부분을 guard문으로 처리
- 디버깅 중 확인한 부분입니다
- (홈 → 상세) 이동 후에도 홈의 배너 무한스크롤이 계속 작동하는 문제
- 홈의 무한 스크롤이 한번 시작했음에도 계속 다시 시작하는 문제
@0Hooni 0Hooni added 🎨 style 코드 스타일 수정, 컨벤션 적용 🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 labels Apr 4, 2025
@0Hooni 0Hooni self-assigned this Apr 4, 2025
@0Hooni 0Hooni linked an issue Apr 4, 2025 that may be closed by this pull request
@0Hooni 0Hooni changed the title 사진이 한장일 때 무한 스크롤을 방지하도록 수정 [FIX] 사진이 한장일 때 무한 스크롤을 방지하도록 수정 Apr 4, 2025
Copy link
Member

@dongglehada dongglehada left a comment

Choose a reason for hiding this comment

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

분기를 통해 수정해주셨군요! 확인 완료하였습니다

@zzangzzangguy
Copy link
Contributor

분기처리를 추가하셔서 해결하시구 스크롤 종료에 가드문을 추가하셨군용! 확인하였습니다 수고하셨어요!!

@0Hooni 0Hooni merged commit 872f289 into dev Apr 5, 2025
@0Hooni 0Hooni deleted the fix/#82-block-swipe-when-carousel-have-one-image branch April 5, 2025 03:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 🎨 style 코드 스타일 수정, 컨벤션 적용

Projects

None yet

Development

Successfully merging this pull request may close these issues.

이미지가 하나인 캐러셀에서 무한 스와이프 제거

4 participants