[WEAV-53] Design System - Core Change #18
Conversation
Walkthrough이번 변경 사항은 디자인 시스템의 색상 자산 및 UI 구성 요소에 대한 여러 수정 사항을 포함합니다. 색상 정의가 업데이트되었으며, 새로운 색상 자산이 추가되었습니다. 또한, SwiftUI 파일에서는 강제 언래핑을 제거하여 안전성을 개선하고, 버튼 및 텍스트 스타일의 타이포그래피가 조정되었습니다. 이러한 변경은 디자인의 일관성을 높이고, 코드의 안전성을 강화하는 데 기여합니다. Changes
Possibly related PRs
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (15)
- Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Blue300.colorset/Contents.json (1 hunks)
- Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Green100.colorset/Contents.json (1 hunks)
- Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Grey50.colorset/Contents.json (1 hunks)
- Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Pink100.colorset/Contents.json (1 hunks)
- Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Yellow100.colorset/Contents.json (1 hunks)
- Projects/DesignSystem/DesignCore/Sources/BackgroundTextureView.swift (2 hunks)
- Projects/DesignSystem/DesignCore/Sources/CTAButton/CTABottomButton.swift (1 hunks)
- Projects/DesignSystem/DesignCore/Sources/CTAButton/CTAButton.swift (1 hunks)
- Projects/DesignSystem/DesignCore/Sources/Toast/ToastView.swift (1 hunks)
- Projects/DesignSystem/DesignCore/Sources/Typography.swift (3 hunks)
- Projects/Features/DesignPreview/Sources/DesignButtonView.swift (1 hunks)
- Projects/Features/DesignPreview/Sources/DesignColorPreview.swift (1 hunks)
- Projects/Features/DesignPreview/Sources/DesignTextInputPreview.swift (4 hunks)
- Projects/Features/DesignPreview/Sources/DesignTypographyPreview.swift (1 hunks)
- Tuist/ResourceSynthesizers/Assets.stencil (1 hunks)
Additional comments not posted (22)
Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Blue300.colorset/Contents.json (1)
8-10: 색상 구성 요소 변경 사항이 승인되었습니다.색상 구성 요소 값에 대한 변경 사항은 이전 버전에 비해 약간 어둡고 채도가 낮은 파란색으로 이어집니다. 이러한 수정 사항은 디자인 시스템의 핵심 요소를 개선하고 시각적 일관성을 향상시키려는 PR 목표에 부합합니다. 색 공간은 "display-p3"로 변경되지 않은 상태로 유지됩니다.
Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Green100.colorset/Contents.json (1)
1-20: 새로운 "Green100" 색상이 디자인 시스템에 잘 추가되었습니다!이 파일은 "Green100"이라는 새로운 색상을 정의하며, 디자인 시스템의 색상 팔레트를 확장하는 데 도움이 됩니다. 색상 공간, RGB 값 및 알파 값이 올바르게 지정되었으며, "universal" 이디엄을 사용하여 다양한 디바이스에서 일관되게 사용할 수 있습니다. 이 새로운 녹색 색조는 UI 요소 및 테마에 대한 더 많은 옵션을 제공할 것입니다.
Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Grey50.colorset/Contents.json (1)
1-20: 변경 사항이 좋아 보입니다!새로운
Grey50색상이 디자인 시스템에 올바르게 추가되었습니다. JSON 파일의 구조와 색상 값이 의도한 대로 설정되어 있습니다. 이 변경 사항은 시각적 일관성을 향상시키는 데 도움이 될 것입니다.Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Pink100.colorset/Contents.json (1)
1-20: 새로운 색상 자산이 디자인 시스템에 잘 추가되었습니다!이 파일은 "Pink100"이라는 새로운 색상 자산을 정의하고 있습니다. 색상은 display-p3 색상 공간을 사용하여 지정되었으며, RGBA 값을 사용하여 색상 구성 요소가 정의되어 있습니다. 또한 이 색상은 universal idiom으로 지정되어 있어 디자인 시스템 색상으로 적절해 보입니다.
새로운 색상 자산을 추가하면 애플리케이션 전체에서 사용할 수 있게 됩니다. 선택된 색상 공간과 색상 구성 요소가 전반적인 디자인 가이드라인 및 기존 색상 팔레트와 일관되는지 확인하는 것이 좋겠습니다.
Projects/DesignSystem/DesignCore/Resources/Colors/Colors.xcassets/Yellow100.colorset/Contents.json (1)
1-20: 색상 리소스 추가를 승인합니다!이 파일은 디자인 시스템에 새로운 Yellow100 색상을 올바르게 정의하고 있습니다. 주요 사항은 다음과 같습니다:
- JSON 구조가 Xcode의 색상 리소스 정의를 따르고 있습니다.
- "display-p3" 색상 공간과 RGBA 컴포넌트가 정확하게 지정되었습니다.
- idiom이 "universal"로 설정되어 다양한 디바이스에서 사용할 수 있습니다.
- 메타데이터(작성자 및 버전)가 포함되어 있습니다.
이 색상은 디자인 시스템 전반에서 사용될 수 있을 것입니다. 변경 사항을 승인합니다.
Projects/DesignSystem/DesignCore/Sources/BackgroundTextureView.swift (2)
15-15: 강제 언래핑 제거 좋습니다!
backgroundDefault이미지에 접근할 때 강제 언래핑을 제거한 것은 런타임 크래시를 방지하는 좋은 방법입니다. 이 변경사항은 코드의 안정성을 높이는 데 도움이 될 것입니다.
30-30: 이 부분에서도 강제 언래핑 제거 잘 되었네요.
BackgroundTextureViewModifier에서도backgroundDefault이미지에 접근할 때 강제 언래핑을 제거했습니다. 이는 이전 변경사항과 일관성을 유지하면서 코드의 안정성을 향상시키는 좋은 방법입니다.Projects/Features/DesignPreview/Sources/DesignButtonView.swift (1)
30-30: 배경 색상 변경이 잘 되었습니다!
CTAButton의backgroundStyle속성을DesignCore.Colors.darkGreen에서DesignCore.Colors.green500으로 변경하여 버튼의 시각적 모양을 개선했습니다. 이 변경은 디자인 시스템의 핵심 요소, 특히 색상 사양을 개선하려는 PR 목표와 일치합니다.Projects/DesignSystem/DesignCore/Sources/CTAButton/CTAButton.swift (1)
48-48: 좋아 보입니다!버튼 텍스트의 타이포그래피 스타일을
.medium_18에서.semibold_18로 변경하는 것은 PR의 목표인 시각적 일관성과 심미성 향상에 부합합니다. 이 변경은 버튼의 동작이나 기능에 영향을 미치지 않으며, 버그나 회귀를 유발할 가능성이 낮습니다.Projects/Features/DesignPreview/Sources/DesignColorPreview.swift (2)
28-34: 잘 추가되었습니다!
lightColors배열이DesignCore.Colors에서 일관되게 색상 값을 가져와 통합된 색상 시스템을 유지하고 있습니다. 배열과 색상 값의 명명 규칙이 명확하고 기존 패턴을 따르고 있어 코드 가독성이 좋습니다.
37-40: 파스텔 색상 업데이트가 잘 되었네요!
pastelColors배열이DesignCore.Colors에서 일관되게 새로운 색상 값을 가져와 통합된 색상 시스템을 유지하고 있습니다. 색상 값의 명명 규칙이 기존 패턴을 따르고 있어 코드 일관성이 좋습니다.Projects/DesignSystem/DesignCore/Sources/CTAButton/CTABottomButton.swift (1)
55-55: 좋아 보입니다!버튼 제목의 글꼴 두께를 medium에서 semibold로 변경하는 것은 가독성과 시각적 두드러짐을 향상시키는 적절한 선택입니다. 이 변경은 디자인 시스템의 전반적인 심미성을 개선하는 데 도움이 될 것입니다.
Projects/Features/DesignPreview/Sources/DesignTypographyPreview.swift (3)
39-39: 색상 변경이 적절해 보입니다.
DesignCore.Colors.darkGreen에서DesignCore.Colors.green500으로의 변경은 디자인 시스템의 색상 팔레트와 일치하며, PR의 목표인 시각적 일관성과 심미성 향상에 부합합니다.
42-42: 이전 변경사항과 일관된 색상 변경입니다.
DesignCore.Colors.darkGreen에서DesignCore.Colors.green500으로의 변경은 이전 코드 세그먼트와 일치하며, 디자인 시스템의 색상 팔레트와도 부합합니다. 이는 PR의 목표인 시각적 일관성과 심미성 향상에 기여할 것으로 보입니다.
47-47: 타이포그래피 스타일 변경이 적절해 보입니다.
.medium_18에서.semibold_18로의 변경은 디자인 시스템의 타이포그래피 사양과 일치하며, PR의 목표인 디자인 시스템의 핵심 요소 개선 및 타이포그래피 향상에 부합합니다.Projects/DesignSystem/DesignCore/Sources/Typography.swift (3)
20-20: 좋아 보입니다!새로운
semibold_18케이스가Typography열거형에 잘 추가되었습니다. 이 변경 사항은 기존 코드와 일관성이 있고 문제가 없어 보입니다.
41-41: 잘 되었습니다!
Typography확장의fontSize계산 속성 내 switch 문에semibold_18케이스를 추가하고 폰트 크기 18을 반환하도록 한 것이 적절해 보입니다. 이 변경 사항은 기존 코드와 잘 어울리고 문제가 없습니다.
66-66: 변경 사항을 검토해 주세요.
Typography확장의pretendardWeight계산 속성에서semibold_18케이스를._600가중치를 반환하는 케이스에 추가하고,medium_18케이스를._500가중치를 반환하는 케이스에서 제거한 것은 적절해 보입니다.하지만
medium_18케이스의 제거가 의도적인 것인지, 그리고 기존 코드를 깨뜨리지 않는지 확인이 필요할 것 같습니다.
medium_18케이스의 제거가 의도적이고 문제가 없는지 확인하기 위해 다음 스크립트를 실행해 보세요:Also applies to: 69-69
Projects/Features/DesignPreview/Sources/DesignTextInputPreview.swift (4)
23-23: 좋은 변경사항입니다!옵셔널 이미지를 강제 언래핑하는 대신 옵셔널을 그대로 할당하는 것은 이미지가
nil일 경우 런타임 크래시를 방지할 수 있는 좋은 방법입니다.
39-39: 이전 변경사항과 일관성 있는 좋은 수정입니다.이전 변경사항과 마찬가지로 옵셔널 이미지를 강제 언래핑하지 않고 할당하는 것은 안전한 코딩 습관입니다.
49-49: 이전 변경사항들과 일관되게 좋은 수정이 이루어졌습니다.이 변경사항 역시 이전 변경사항들과 마찬가지로 옵셔널 이미지를 안전하게 할당하고 있습니다. 좋은 코딩 습관이 잘 반영된 것 같습니다.
61-61: 파일 전체적으로 일관성 있는 좋은 수정입니다.이 파일의 모든 변경사항이 옵셔널 이미지를 안전하게 할당하도록 수정되었습니다. 전반적으로 매우 좋은 코딩 습관이 반영된 것 같습니다.
구현사항
Summary by CodeRabbit
New Features
Green100,Grey50,Pink100,Yellow100lightColors추가 및pastelColors업데이트Bug Fixes
Style
semibold_18로 업데이트Chores
Blue300색상 조정Refactor