Skip to content

[design] #97 앱 아이콘 교체#98

Merged
Ojongseok merged 1 commit into
developfrom
design/#97-app-icon
Feb 10, 2026
Merged

[design] #97 앱 아이콘 교체#98
Ojongseok merged 1 commit into
developfrom
design/#97-app-icon

Conversation

@Ojongseok
Copy link
Copy Markdown
Member

@Ojongseok Ojongseok commented Feb 10, 2026

🔗 관련 이슈

📙 작업 설명

  • 앱 아이콘 변경

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

기능 미리보기 기능 미리보기
기능 설명

Summary by CodeRabbit

릴리스 노트

  • 스타일
    • 앱 런처 아이콘의 리소스 구성을 개선하고 아이콘 렌더링을 최적화했습니다.

@Ojongseok Ojongseok requested a review from ikseong00 February 10, 2026 10:07
@Ojongseok Ojongseok self-assigned this Feb 10, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 10, 2026

📋 개요

앱 아이콘 구성 파일들이 업데이트되어 플레이스토어 심사 제출을 위한 아이콘 리소스 참조가 변경되었습니다. 백그라운드 XML의 구조가 재정렬되고, 어댑티브 아이콘 설정에서 포어그라운드 리소스가 drawable에서 mipmap으로 변경되었으며 monochrome 설정이 제거되었습니다.

📊 변경 사항

Cohort / File(s) Summary
배경 이미지 정의
app/src/main/res/drawable/ic_launcher_background.xml
XML 네임스페이스 선언 이동, viewportHeight 및 viewportWidth 속성 추가, 벡터 태그 속성 재정렬 및 재구성(70행 추가, 166행 제거)
어댑티브 아이콘 설정
app/src/main/res/mipmap-anydpi-v26/ic_launcher.xml, app/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml
포어그라운드 리소스 참조를 @drawable/ic_launcher_foreground에서 @mipmap/ic_launcher_foreground로 변경, monochrome 태그 제거, 마이너 포매팅 정리

⏱️ 코드 리뷰 예상 소요 시간

🎯 2 (Simple) | ⏱️ ~8 minutes

🐰 축하 시

아이콘이 새 옷을 입고 반짝반짝해졌네요, 🎨
드래워블에서 밉맵으로 우아하게 옮겨가고,
플레이스토어 심사는 이제 코앞이라네요! ✨
적응형 아이콘이 춤을 추며 빛나고,
우리의 앱이 더욱 돋보일 날이 왔어요! 🚀

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 앱 아이콘 교체라는 주요 변경사항을 명확하게 설명하며, 간결하고 구체적입니다.
Linked Issues check ✅ Passed PR의 모든 변경사항(아이콘 관련 XML 파일 수정)은 #97 이슈의 '앱 아이콘 변경' 목적을 충족합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 앱 아이콘 업데이트 범위 내에 있으며, 요구사항 범위를 벗어난 변경은 없습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch design/#97-app-icon

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

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@app/src/main/res/drawable/ic_launcher_background.xml`:
- Around line 2-74: Replace the background fillColor used in the vector
drawable's first path (the element with android:pathData="M0,0h108v108h-108z"
and android:fillColor="#3DDC84") with the PR/design's teal/teal‑green hex;
update the android:fillColor attribute value there (replace "#3DDC84" with the
correct design hex) so the adaptive icon background matches the design.

Comment thread app/src/main/res/drawable/ic_launcher_background.xml
Copy link
Copy Markdown
Contributor

@ikseong00 ikseong00 left a comment

Choose a reason for hiding this comment

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

Good!!

@Ojongseok Ojongseok merged commit d376986 into develop Feb 10, 2026
5 checks passed
@Ojongseok Ojongseok deleted the design/#97-app-icon branch February 10, 2026 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] 앱 아이콘 변경

2 participants