Skip to content

Conversation

@thingineeer
Copy link
Member

@thingineeer thingineeer commented Mar 17, 2025

🩵 Issue

close #277


💙 변경된 내용

  • 홈, 공고상세, 검색 뷰 placeHolder 추가
  • setImage 함수 중복 로직 수정

🅿️ PR Point

  • 아래 처럼 사용하는 부분에 placeHolder를 달아주면 각 Cell을 사용하고 있는 뷰는 placeHolder가 적용이됩니다.
self.jobCardCoverImage.setImage(with: model.companyImage, placeholder: "img_post_placeHolder")

📘 ScreenShot

  • 시뮬레이션에는 일부러 GCD 시간초를 둬서 이미지가 늦게 로딩되게 했습니다.

@thingineeer thingineeer added 명진 🍀 Feat ✨ 새로운 기능 구현 labels Mar 17, 2025
@thingineeer thingineeer requested a review from wjdalswl March 17, 2025 15:07
@thingineeer thingineeer self-assigned this Mar 17, 2025
@thingineeer thingineeer changed the title [Feat] #277 홈 뷰, 공고 상세 뷰 ,검색 뷰 플레이스홀더 구현 [Feat] #277 홈 뷰, 공고 상세 뷰 ,검색 뷰 플레이스홀더 구현 했습니다. Mar 18, 2025
Copy link
Member

@wjdalswl wjdalswl left a comment

Choose a reason for hiding this comment

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

확인했슴니다! 수고하셨어요~~

extension CompanyInfoTableViewCell {
func bind(with companyInfo: CompanyInfoModel) {
companyImageView.setImage(with: companyInfo.companyImage ?? "placeholder_image", placeholder: "placeholder_image")
companyImageView.setImage(with: companyInfo.companyImage ?? "placeholder_image", placeholder: "img_post_placeHolder")
Copy link
Member

Choose a reason for hiding this comment

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

companyInfo.companyImage 값이 없을때도 "placeholder_image"말고 "img_post_placeHolder" 써도 되지 않을까요??

Copy link
Member Author

Choose a reason for hiding this comment

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

값이 없을 때 <- 빈 이미지
값이 뜨기 전 <- 플레이 홀 더 이미지

이렇게 분리해서 생각을 했는데, 통일할지는 디자인 선생님들에게 물어봐야 할 것 같네요!

override func prepareForReuse() {
super.prepareForReuse()

self.jobCardCoverImage.image = UIImage(resource: .imgPostPlaceHolder)
Copy link
Member

Choose a reason for hiding this comment

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

이러면 이제 이미지 밀리는거 해결 되나요?

Copy link
Member Author

Choose a reason for hiding this comment

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

prepareForReuse() 호출할 때, 기본값을 넣어주면 해결되더라고요!
더 좋은 방법이 있는지는 공부하면서 알아봐야 할 것 같습니다.

@thingineeer thingineeer changed the title [Feat] #277 홈 뷰, 공고 상세 뷰 ,검색 뷰 플레이스홀더 구현 했습니다. [Feat] #277 - 홈 뷰, 공고 상세 뷰 ,검색 뷰 플레이스홀더 구현 했습니다. Mar 19, 2025
@thingineeer thingineeer merged commit 9688ead into main Mar 19, 2025
@thingineeer thingineeer deleted the Feat/#277-홈_상세_플레이스홀더 branch March 19, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feat ✨ 새로운 기능 구현 명진 🍀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] 홈, 공고 상세 로딩 뷰 구현

3 participants