style: Atomic Green Color값 수정 및 적용 #224
Conversation
Walkthrough디자인 시스템 컬러 값 일부를 갱신하고, 런치 스크린 배경색과 이미지 위치를 조정했다. 내비게이션 바 타이틀 이미지에 틴트 색을 설정했다. 작은 로고의 Contents.json에 template 렌더링 속성을 추가했다. 코드 구조나 공개 API 시그니처 변경은 없다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10–15 minutes Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/Projects/BKPresentation/Sources/Common/Extension/UINavigationController+.swift (1)
327-327: 배경색을 tint로 사용하는 것이 의도된 것인지 확인 필요.
imageView.tintColor에.bkBackgroundColor(.primary)를 사용하고 있습니다. 일반적으로 tintColor는 콘텐츠/전경색(.bkContentColor)을 사용하는 것이 의미론적으로 적절합니다.디자인 시스템에서 배경색을 틴트로 사용하도록 의도한 것이 맞는지 확인해 주세요.
만약 콘텐츠 색상을 사용해야 한다면 다음과 같이 수정을 고려해보세요:
- imageView.tintColor = .bkBackgroundColor(.primary) + imageView.tintColor = .bkContentColor(.brand)
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
src/Projects/BKDesign/Resources/Assets.xcassets/Logos/smallLogo.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Sources/Foundation/ColorSystem/BKAtomicColor.swift(1 hunks)src/Projects/BKPresentation/Sources/Common/Extension/UINavigationController+.swift(1 hunks)src/Projects/Booket/Resources/LaunchScreen.storyboard(2 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/Projects/BKPresentation/Sources/Common/Extension/UINavigationController+.swift (1)
src/Projects/BKDesign/Sources/Extensions/UIColor+.swift (1)
bkBackgroundColor(38-50)
⏰ 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 (9)
src/Projects/BKDesign/Resources/Assets.xcassets/Logos/smallLogo.imageset/Contents.json (1)
21-24: 템플릿 렌더링 설정 추가 확인.로고 이미지에 template rendering intent를 추가하여 tint color 적용이 가능하도록 설정되었습니다. UINavigationController의 tintColor 설정과 올바르게 연동됩니다.
src/Projects/Booket/Resources/LaunchScreen.storyboard (3)
2-2: Xcode 도구 버전 업데이트 확인.storyboard의 toolsVersion과 plugIn 버전이 업데이트되었습니다. 이는 Xcode 버전 업그레이드에 따른 자동 변경입니다.
Also applies to: 6-6
20-20: 이미지 위치 조정 확인.imageView의 y 좌표가 미세하게 조정되었습니다(367 → 379.333...). 센터링 제약조건으로 인한 자동 계산 결과로 보입니다.
24-24: 런치 스크린 배경색이 Atomic Green g500과 일치합니다.
RGB(0.231, 0.761, 0.357) ≈ #3BC25B → BKAtomicColor.Green.g500 확인 완료.src/Projects/BKDesign/Sources/Foundation/ColorSystem/BKAtomicColor.swift (3)
40-47: Green 팔레트 hex 값 업데이트 완료
BKAtomicColor.Green의 g200~g900 값이 디자인 스펙에 맞게 변경되었으며, BKSemanticColor 등 호출부에서도 일관적으로 반영됨을 확인했습니다.
40-47: Green 팔레트 hex 업데이트 확인
g200~g900 hex값이 올바르게 반영되었으며, 전역 하드코딩된 이전 색상 참조는 모두 제거되었습니다.
- WCAG 명암비 기준 충족 여부를 검증해주세요.
40-47: 직접BKAtomicColor.g200~g900사용처 미발견
직접 참조가 없으므로 semantic 레이어(BKSemanticColor) 매핑에 변경된 Green 값이 올바르게 반영되었는지 확인하고, UI 전반에서 시각적 검증을 진행해 주세요.src/Projects/BKPresentation/Sources/Common/Extension/UINavigationController+.swift (2)
327-327: 이미지 tintColor 색상 시맨틱 의도 확인
bkBackgroundColor(.primary)를tintColor에 사용하는 것이 의도된 디자인 결정인지 확인- smallLogo asset에
template-rendering-intent: template이 설정되어 있어 tint 적용은 정상입니다
327-327: tintColor에 background 색상 대신 content semantic 색상 사용 검토
src/Projects/BKPresentation/Sources/Common/Extension/UINavigationController+.swift (makeHomeImageTitleView, line 327)에서.bkBackgroundColor(.primary)를 사용 중인데, 코드베이스 전반에서는 tintColor에.bkContentColor계열을 사용합니다. 의도된 디자인 결정이 아니라면.bkContentColor(.primary)또는 브랜딩 녹색이 필요할 경우.bkContentColor(.brand)로 변경해 주세요.
🔗 관련 이슈
📘 작업 유형
📙 작업 내역
🧪 테스트 내역
🎨 스크린샷 또는 시연 영상 (선택)
✅ PR 체크리스트
💬 추가 설명 or 리뷰 포인트 (선택)
Summary by CodeRabbit