로그인 & 통계 상세 화면 일러스트 변경

[chanho0908]

이슈 번호

작업내용

• 로그인 화면 디자인 수정
• 통계 상세 화면 일러스트 수정

결과물

로그인

Before	After

통계 화면

Before	After

리뷰어에게 추가로 요구하는 사항 (선택)

[coderabbitai]

📝 Walkthrough

Walkthrough

이 변경사항은 디자인 시스템의 벡터 드로어블 리소스를 정리하는 PR입니다. 기존 ic_hug.xml의 경로 데이터, 색상값, 획 너비를 여러 섹션에서 조정하였으며, 새로운 ic_singing.xml 벡터 드로어블을 추가했습니다. 또한 LoginScreen.kt의 이미지 참조를 기존 ic_keepi_singing에서 새로운 ic_singing으로 업데이트했습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

---

상세 검토 코멘트

전반적인 평가

벡터 드로어블 리소스를 체계적으로 정리하고 새로운 리소스를 추가한 점은 긍정적입니다. 다만 몇 가지 검토 포인트가 있습니다.

🔍 ic_hug.xml 수정사항 검토

상황 분석:
pathData 좌표, 색상값(#FFD5DE → #FDE0DF, #ffffff → #FDE0DF), 획 너비(1.2 → 1.11751)가 대규모로 변경되었습니다. 이러한 수정이 의도적인 디자인 변경인지 확인이 필요합니다.

질문:

• 이 변경사항들이 디자인팀의 승인을 받은 수정사항인가요? 시각적 회귀 테스트가 수행되었나요?
• 색상 변경(#FFD5DE → #FDE0DF)의 목적이 무엇인지 설명해주실 수 있을까요?

개선 제안:
각 주요 변경사항에 주석을 추가하여 의도를 명확히 하면, 향후 유지보수 시 이해도가 높아질 것 같습니다.

🎨 ic_singing.xml 추가 검토

칭찬할 점:

• 새로운 벡터 드로어블이 일관된 구조로 잘 정의되었습니다.
• 채우기 색상, 획 색상, 획 너비 등이 명확히 지정되어 있습니다.

검토 포인트:
IC_singing.xml의 326줄은 상당한 규모입니다.

제안:

• 복잡한 경로들이 여러 개 있다면, 로직적으로 관련된 경로들을 그룹화하는 것을 고려해보세요.
• 반복되는 색상값들(예: #FDE0DF, #4A4A4A)을 색상 리소스로 추출하면, 향후 테마 변경 시 유지보수가 더 수월합니다.

예시:

<!-- colors.xml -->
<color name="singing_primary">#FDE0DF</color>
<color name="singing_secondary">#4A4A4A</color>

📝 LoginScreen.kt 참조 업데이트

평가:
리소스 참조 업데이트는 명확하고 직관적입니다. ic_keepi_singing → ic_singing으로의 변경이 기존 리소스 제거와 함께 수행되었나요?

질문:

• 기존 ic_keepi_singing 리소스가 다른 곳에서도 사용되고 있지는 않나요? 안전하게 제거할 수 있는지 확인이 필요합니다.

---

종합 의견

이 PR은 디자인 시스템을 정리하려는 좋은 의도가 보입니다. 다만 대규모 드로어블 수정과 신규 추가가 동시에 이루어지고 있으므로, 시각적 검증과 색상 리소스 재사용성을 함께 고려하면 더욱 강력한 변경사항이 될 것 같습니다.

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)

Check name	Status	Explanation
Description check	✅ Passed	PR 설명은 한국어로 작성되었으며 로그인 화면 및 통계 화면 일러스트 수정에 대해 명확히 설명하고, Before/After 스크린샷으로 변경사항을 시각적으로 보여줍니다.
Title check	✅ Passed	PR 제목이 변경 사항의 주요 내용을 명확하게 반영합니다. 로그인 및 통계 상세 화면의 일러스트 변경이라는 핵심 목표를 잘 표현하고 있습니다.

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

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/#114-illust-fix

---

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