Skip to content

[Refactor] 네트워크 기본 이미지 변경#159

Merged
kimjw2003 merged 1 commit intodevelopfrom
refactor/#158-networkimage-default
Jan 21, 2026
Merged

[Refactor] 네트워크 기본 이미지 변경#159
kimjw2003 merged 1 commit intodevelopfrom
refactor/#158-networkimage-default

Conversation

@nahy-512
Copy link
Copy Markdown
Contributor

@nahy-512 nahy-512 commented Jan 21, 2026

📮 관련 이슈

📌 작업 내용

  • 기본 이미지 변경하고, NetworkImage에서 loading, error 시의 Painter 뚫어줬어요.

📸 스크린샷

작품 탐색 컬렉션 파일
Screenshot_20260121_231531 Screenshot_20260121_231537 Screenshot_20260121_231551

🫛 To. 리뷰어

  • 디샘들께 이미지 받아서 모든 곳에 공통적으로 쓰게 해놨습니다

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 네트워크 이미지 로딩 중 및 오류 시 표시할 이미지를 사용자 정의할 수 있습니다.
  • 스타일

    • 컬렉션 아이템 배경 스타일을 개선했습니다.
    • UI 요소의 패딩 및 스타일링을 미세 조정했습니다.

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

@nahy-512 nahy-512 requested a review from a team as a code owner January 21, 2026 14:18
@nahy-512 nahy-512 linked an issue Jan 21, 2026 that may be closed by this pull request
2 tasks
@nahy-512 nahy-512 added the Refactor ♻️ 기능 변경 없이 코드 구조를 개선하여 가독성, 유지보수성, 성능을 향상 label Jan 21, 2026
@nahy-512 nahy-512 self-assigned this Jan 21, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 21, 2026

📝 Walkthrough

개요

NetworkImage 컴포넌트에 placeholder 및 error Painter 매개변수를 추가하고, CollectionItem과 CollectionFileItem에서 배경 스타일을 제거 또는 조정했습니다.

변경사항

코호트 / 파일 변경 요약
NetworkImage Painter 매개변수 추가
app/src/main/java/com/flint/core/designsystem/component/image/NetworkImage.kt
placePaintererrorPainter 매개변수 추가 (기본값: painterResource(R.drawable.img_network_loading)); ColorPainter에서 Painter로 import 변경; AsyncImage 호출에서 placeholder 및 error 상태 적용
배경 스타일 제거
app/src/main/java/com/flint/core/designsystem/component/listItem/CollectionItem.kt
Box modifier 체인에서 FlintTheme.colors.gray200 배경 modifier 제거
Modifier 포맷팅 및 배경 조정
app/src/main/java/com/flint/presentation/collectionlist/component/CollectionFileItem.kt
CollectionFileContent와 CollectionPocketPoster에서 modifier 포맷팅 정리; CollectionPocketPoster에서 명시적 Color.White 배경 제거; 불필요한 trailing comma 정리

예상 코드 리뷰 난이도

🎯 2 (단순) | ⏱️ ~10분

🐰✨ NetworkImage에 새로운 화면을 입혔고,

배경은 깔끔하게 정리하네요.

Painter와 modifier, 제 자리를 찾아

디자인 시스템이 한층 곱아졌어요! 🎨

🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 PR의 주요 변경사항인 네트워크 기본 이미지 변경을 명확하게 설명하고 있습니다.
Description check ✅ Passed PR 설명이 관련 이슈, 작업 내용, 스크린샷, 리뷰어 메모를 포함하여 템플릿 요구사항을 충족합니다.
Linked Issues check ✅ Passed PR은 이슈 #158의 모든 주요 목표를 충족합니다: 이미지 교체 및 NetworkImage에 placeholder/error 필드 추가.
Out of Scope Changes check ✅ Passed 모든 변경사항이 이슈 #158의 범위 내에 있으며, NetworkImage 컴포넌트 개선 및 관련 UI 조정만 포함됩니다.

✏️ 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

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

Copy link
Copy Markdown
Contributor

@giovannijunseokim giovannijunseokim left a comment

Choose a reason for hiding this comment

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

LGTM 💯

@kimjw2003 kimjw2003 merged commit e079997 into develop Jan 21, 2026
2 checks passed
@kimjw2003 kimjw2003 deleted the refactor/#158-networkimage-default branch January 21, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Refactor ♻️ 기능 변경 없이 코드 구조를 개선하여 가독성, 유지보수성, 성능을 향상

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] NetworImage 기본 이미지 수정

3 participants