feat: thumbnail image에 border 추가

[doyeonk429]

🔗 관련 이슈

• Close [BOOK-429/qa] [iOS] 책 커버 라인 추가 #257

📘 작업 유형

• ✨ Feature (기능 추가)
• 🐞 Bugfix (버그 수정)
• 🔧 Refactor (코드 리팩토링)
• ⚙️ Chore (환경 설정)
• 📝 Docs (문서 작성 및 수정)
• ✅ Test (기능 테스트)
• 🎨 style (코드 스타일 수정)

📙 작업 내역

• 책 표지 썸네일에 border 추가

🧪 테스트 내역

• 브라우저/기기에서 동작 확인
• 엣지 케이스 테스트 완료
• 기존 기능 영향 없음

🎨 스크린샷 또는 시연 영상 (선택)

✅ PR 체크리스트

• 커밋 메시지가 명확합니다
• PR 제목이 컨벤션에 맞습니다
• 관련 이슈 번호를 작성했습니다
• 기능이 정상적으로 작동합니다
• 불필요한 코드를 제거했습니다

💬 추가 설명 or 리뷰 포인트 (선택)

• ..

Summary by CodeRabbit

• 스타일
  • 썸네일 이미지에 테두리 스타일이 추가되었습니다.

[coderabbitai]

Walkthrough

두 개 파일에 책 커버 썸네일에 시각적 테두리를 추가했습니다. BKBookSummaryView에는 테두리 너비와 색상을 제어하는 계산 속성을 도입했고, HomeCardCell에는 테두리 스타일을 적용했습니다.

Changes

집단 / 파일명	변경 요약
BKBookSummaryViewStyle 테두리 스타일 속성 src/Projects/BKDesign/Sources/Components/Summary/BKBookSummaryView.swift	thumbnailBorderWidth 계산 속성 추가 (기본값 1.0, alreadyEnroll 스타일일 경우 0.0) 및 thumbnailBorderColor 계산 속성 추가 (기본값 primary 테두리 색상, alreadyEnroll일 경우 투명). BKBookSummaryView의 configure 메서드에서 이 속성들을 thumbnail 레이어 테두리에 적용
HomeCardCell 썸네일 테두리 스타일링 src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeCardCell.swift	configure 메서드에서 썸네일 이미지 뷰에 borderWidth 1.0과 primary BK 테두리 색상으로 설정. UI 스타일링 조정만 포함

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• 두 파일 모두 동일한 패턴의 일관된 스타일링 변경
• 새로운 계산 속성은 조건부 값 반환으로 구현되어 로직이 단순함
• 제어 흐름 변경 없이 순수 UI 스타일 조정만 포함

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	연결된 이슈 #257에서 요청한 4개 화면(홈, 내 서재, 검색, 도서 상세) 중 홈과 책 상세 화면에 대해서만 border 추가가 구현되었습니다.	내 서재 및 검색 화면의 책 표지 썸네일에도 border를 추가하여 모든 요청 사항을 완료해야 합니다.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경 사항의 주요 내용과 일치합니다. 썸네일 이미지에 border를 추가하는 기능 변경을 명확하게 설명하고 있습니다.
Out of Scope Changes check	✅ Passed	변경 사항이 모두 요청된 border 추가 기능에 관련된 내용이며, 요청 범위 내의 UI 스타일 조정입니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch BOOK-429-fix/#257

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dcc0821 and 5c4972a.

📒 Files selected for processing (2)

• src/Projects/BKDesign/Sources/Components/Summary/BKBookSummaryView.swift (2 hunks)
• src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeCardCell.swift (1 hunks)

🧰 Additional context used

🧠 Learnings (3)

📓 Common learnings

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 157
File: src/Projects/BKPresentation/Sources/AuthFlow/View/LoginView.swift:43-45
Timestamp: 2025-08-08T01:39:15.620Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 접근성 개선 작업을 별도의 전용 PR이나 이슈에서 일괄 처리하는 것을 선호한다. 개별 기능 구현 PR에서는 접근성 관련 제안을 하지 않고, 접근성 전담 작업에서 한번에 처리하는 방식을 원한다.

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 157
File: src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeViewController.swift:19-24
Timestamp: 2025-08-08T01:38:59.656Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 접근성(accessibility) 관련 개선사항은 현재 작업 중인 PR에서 즉시 처리하지 않고, 접근성 전용 PR이나 이슈를 별도로 만들어 한번에 처리하는 것을 선호한다.

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 68
File: src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonTestViewController.swift:124-133
Timestamp: 2025-07-10T08:21:49.399Z
Learning: doyeonk429는 테스트 전용으로만 사용되는 extension이나 코드는 해당 테스트 파일에 그대로 두는 것을 선호합니다. 실제 프로덕션 코드에서 사용되지 않는 테스트 전용 코드는 별도 파일로 분리하지 않고 테스트 파일 내에 유지하는 것이 그들의 코드 구성 방식입니다.

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 138
File: src/Projects/BKDesign/Resources/Assets.xcassets/icons/home.imageset/Contents.json:4-14
Timestamp: 2025-08-04T15:20:43.982Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 에셋 파일명이 영어가 아닌 경우(한글, 일본어 등)에는 경고를 받고 싶어하지만, 영어 파일명의 네이밍 컨벤션(예: home.png vs home1.png)은 기능적으로 문제없다면 신경쓰지 않는 것을 선호한다.

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 98
File: src/Projects/BKPresentation/Sources/AuthFlow/View/TermsView.swift:78-85
Timestamp: 2025-07-22T05:37:28.756Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 더미 데이터나 테스트 데이터의 구체적인 내용(예: URL 주소)에 대해서는 리뷰 시 지적하지 않기를 선호한다. 더미 데이터는 임시적이며 실제 기능 구현 시 교체될 예정이므로 URL의 구체적인 주소는 중요하지 않다고 본다.

📚 Learning: 2025-08-20T04:50:43.364Z

Learnt from: clxxrlove
Repo: YAPP-Github/Reed-iOS PR: 181
File: src/Projects/BKDesign/Sources/Components/TextField/BKTextView.swift:183-186
Timestamp: 2025-08-20T04:50:43.364Z
Learning: In BKTextView (src/Projects/BKDesign/Sources/Components/TextField/BKTextView.swift), the border width is intentionally kept at LayoutConstants.borderWidth even when there's no error message, with only the color changed to clear. This maintains layout consistency by preserving the border space.

Applied to files:

• src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeCardCell.swift
• src/Projects/BKDesign/Sources/Components/Summary/BKBookSummaryView.swift

📚 Learning: 2025-08-11T12:49:16.942Z

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 169
File: src/Projects/BKPresentation/Sources/MainFlow/Search/View/SearchResultCell.swift:31-41
Timestamp: 2025-08-11T12:49:16.942Z
Learning: In Reed-iOS project's SearchResultCell, a new BKBookSummaryView instance is created each time configure() is called rather than reusing the existing view. The old view is properly removed and set to nil in prepareForReuse(), allowing deinitialization to handle all cleanup including image downloads. Calling clearView() before removal is unnecessary in this pattern.

Applied to files:

• src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeCardCell.swift

🧬 Code graph analysis (2)

src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeCardCell.swift (1)

src/Projects/BKDesign/Sources/Extensions/UIColor+.swift (1)

• bkBorderColor (65-76)

src/Projects/BKDesign/Sources/Components/Summary/BKBookSummaryView.swift (1)

src/Projects/BKDesign/Sources/Extensions/UIColor+.swift (1)

• bkBorderColor (65-76)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build

🔇 Additional comments (3)

src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeCardCell.swift (1)

102-104: LGTM! 썸네일 border 추가가 잘 구현되었습니다.

흰색 배경의 책 커버 가시성을 개선하기 위한 border 스타일이 올바르게 적용되었습니다. cornerRadius 설정과 동일한 위치에서 처리되어 일관성도 좋습니다.

src/Projects/BKDesign/Sources/Components/Summary/BKBookSummaryView.swift (2)

44-61: 잘 설계된 스타일 중앙화입니다.

BKBookSummaryViewStyle에 border 관련 속성을 추가하여 스타일별로 일관되게 관리할 수 있도록 구현하신 점이 좋습니다. 특히 alreadyEnroll 케이스에서 border를 제거한 것이 적절합니다(커버 오버레이가 있기 때문에).

---

272-273: LGTM! Border 스타일이 올바르게 적용되었습니다.

스타일 enum의 계산 속성을 사용하여 border를 적용한 점이 깔끔합니다. 다른 layer 속성들과 함께 논리적인 위치에 배치되어 있습니다.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}