[Fix] 컬렉션 상세 화면 QA

[giovannijunseokim]

📮 관련 이슈

• closed [Fix] 컬렉션 상세 화면 QA 반영 #187

📌 작업 내용

📸 스크린샷

스크린샷

😅 미구현

• [ ]

🫛 To. 리뷰어

Summary by CodeRabbit

릴리스 노트

• UI 개선
  • 컬렉션 상세 페이지의 레이아웃 간격을 최적화했습니다.
  • 저장 버튼의 동작을 단순화하여 북마크 상태에만 기반하도록 개선했습니다.
  • 아이콘 버튼의 내부 여백을 조정하여 더 나은 UI 일관성을 제공합니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

개요

이 PR은 컬렉션 상세 기능을 리팩터링하여 DTO-to-모델 매핑 로직을 전용 매퍼 파일로 이동하고, 도메인 모델과 저장소에서 userId 의존성을 제거하며, UI에서 isMine 파라미터를 제거하고 버튼 패딩을 커스터마이징 가능하게 만듭니다.

변경사항

코호트 / 파일(들)	변경 요약
버튼 컴포넌트 개선 app/src/main/java/com/flint/core/designsystem/component/button/FlintIconButton.kt	contentPadding 파라미터(기본값 10.dp) 추가로 내부 패딩 커스터마이징 활성화
도메인 모델 구조 재정비 app/src/main/java/com/flint/domain/mapper/collection/CollectionDetailMapper.kt	CollectionDetailResponseDto를 CollectionDetailModelNew로 변환하는 새 확장 함수 추가, 중첩된 Author/Content DTO 매핑 포함
	app/src/main/java/com/flint/domain/model/collection/CollectionDetailModel.kt
	app/src/main/java/com/flint/domain/repository/CollectionRepository.kt
UI 화면 업데이트 app/src/main/java/com/flint/presentation/collectioncreate/CollectionCreateScreen.kt	FlintIconButton 사용 시 contentPadding 파라미터 추가
	app/src/main/java/com/flint/presentation/collectiondetail/CollectionDetailScreen.kt

예상 코드 검토 난도

🎯 3 (Moderate) | ⏱️ ~20분

연관 PR

• [Feat] 탐색 화면 셀프 QA #168: CollectionDetailScreen의 targetImageUrl 파라미터 추가/전달 관련 수정
• [Feat] CollectionDetailScreen 추가 구현 #153: CollectionDetailScreen, CollectionDetailModel, 저장소 매핑/소유권 로직 동일 영역 수정
• [Feat] collection create empty UI #102: FlintIconButton API 수정 (contentPadding/enabled 파라미터 추가)

제안 라벨

Fix 🐛, Refactor ♻️

제안 검수자

• nahy-512

축하 시

🐰 마법의 펜으로 userId를 제거하니,
매퍼가 나타나 DTO를 변환하네,
버튼 패딩은 마음대로,
화면은 더욱 깔끔해지고,
컬렉션 상세도 새롭게 태어났어! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (1 warning, 2 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 21.43% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	PR 설명이 템플릿을 따르지만 '작업 내용' 섹션이 비어있고, 스크린샷과 리뷰어 노트도 미작성 상태입니다.	'작업 내용' 섹션에 변경 사항에 대한 구체적인 설명을 추가하고, 실제 스크린샷과 리뷰어를 위한 노트를 작성해주세요.
Linked Issues check	❓ Inconclusive	연결된 이슈 #187은 구체적인 작업 항목이나 체크리스트 없이 제목만 있어 요구사항이 명확하지 않습니다. 그러나 PR의 변경 사항들(isMine 제거, contentPadding 추가, 간격 조정)이 QA 개선과 관련성이 있습니다.	연결된 이슈의 상세 요구사항이 불명확하므로, PR 설명에 각 변경사항이 어떤 QA 문제를 해결하는지 구체적으로 문서화해주세요.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목은 '컬렉션 상세 화면 QA'로 변경 사항의 주요 목적을 명확하게 반영하고 있습니다.
Out of Scope Changes check	✅ Passed	ContentPadding 파라미터 추가, userId/isMine 제거, PreferencesManager 의존성 제거 등의 변경사항들이 모두 컬렉션 상세 화면과 관련 컴포넌트에 국한되어 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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.}

[nahy-512]

LGTM