Skip to content

Conversation

@dongglehada
Copy link
Member

📌 이슈

✅ 작업 사항

  • 샐 재사용 시 로드중인 URL 확인 로직 추가로 재사용 시 이미지 섞임문제 방어
  • 캐싱 히트 시점 extension내 추가하여 처리 속도 개선

@dongglehada dongglehada added the 🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 label May 4, 2025
@dongglehada dongglehada self-assigned this May 4, 2025
@dongglehada dongglehada linked an issue May 4, 2025 that may be closed by this pull request
Copy link
Member

@0Hooni 0Hooni left a comment

Choose a reason for hiding this comment

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

LGTM!!

수고하셨습니다☺️

Comment on lines +9 to +17
private var currentImageURL: String? {
get { objc_getAssociatedObject(self, &currentURLKey) as? String }
set { objc_setAssociatedObject(self, &currentURLKey, newValue, .OBJC_ASSOCIATION_RETAIN_NONATOMIC) }
}

private var placeholderImage: UIImage? {
get { objc_getAssociatedObject(self, &placeholderImageKey) as? UIImage }
set { objc_setAssociatedObject(self, &placeholderImageKey, newValue, .OBJC_ASSOCIATION_RETAIN_NONATOMIC) }
}
Copy link
Member

Choose a reason for hiding this comment

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

연산프로퍼티에서 사용된 메서드 잘 이해가 안되긴하네요 ㅠㅠ

어떻게 동작되는지 회의때 한번 얘기주시면 감사하겠습니다!!

@dongglehada dongglehada changed the title [fix] 이미지 블링크 문제 개선 [FIX] 이미지 블링크 문제 개선 May 5, 2025
Copy link
Contributor

@zzangzzangguy zzangzzangguy left a comment

Choose a reason for hiding this comment

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

LGTM!

@dongglehada dongglehada merged commit 0799e49 into develop May 7, 2025
2 checks passed
@dongglehada dongglehada deleted the fix/#132-fix-imageBlink branch May 7, 2025 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

이미지 뷰 깜빡임 문제 해결!!

4 participants