위젯 추가 설정 추가 #191
위젯 추가 설정 추가 #191
Conversation
Walkthrough이번 변경사항은 Lottie 패키지(lottie-spm)의 추가와 함께, 위젯 관련 기능 모듈의 도입 및 확장을 포함합니다. 패키지 의존성에 Lottie가 추가되었으며, 위젯 기능(AddWidgetFeature)이 새롭게 구현되어 코어 로직, SwiftUI 뷰, 컴포넌트, 상수 및 데이터 모델이 추가되었습니다. 또한, SettingsFeature에 위젯 관리 기능이 통합되고, 앱 초기화, 버전 업데이트, 공유 모듈 및 디자인 시스템 이미지 자산과 같이 여러 모듈 전반에 걸친 사소한 수정이 이루어졌습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant U as 유저
participant SV as SettingsView
participant SC as SettingsCore
participant AW as AddWidgetView
participant AC as AddWidgetCore
U->>SV: 위젯 추가 버튼 탭
SV->>SC: addWidgetButtonDidTap 액션 전송
SC->>AC: addWidgetCore 상태 초기화
SC->>SV: 상태 업데이트로 네비게이션 트리거
SV->>AW: AddWidgetView로 이동
AW->>AC: 위젯 관련 액션 디스패치
AC->>AC: 상태 전환 및 업데이트 처리
Possibly related PRs
Suggested labels
Poem
✨ Finishing Touches
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. 🪧 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 (
|
✅ Successful finished SwiftLint |
✅ PR의 Assign 자동 지정을 성공했어요! |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 6
🧹 Nitpick comments (13)
Projects/Feature/AddWidgetFeature/Sources/AddWidgetView.swift (4)
7-7: SwiftLint 디렉티브 설정 수정 필요SwiftLint 비활성화 지시문이 특정 규칙을 명시하지 않고 있습니다. 비활성화할 특정 규칙을 명시하여 더 명확하게 처리하세요.
- // swiftlint: disable + // swiftlint: disable:next file_length또는 비활성화하려는 규칙에 맞게 변경하세요.
🧰 Tools
🪛 SwiftLint (0.57.0)
[Warning] 7-7: swiftlint command does not specify any rules
(invalid_swiftlint_command)
107-116: 가이드 텍스트의 지역화(로컬라이제이션) 필요가이드 텍스트가 하드코딩되어 있습니다. 다국어 지원을 위해 지역화(로컬라이제이션)를 구현하는 것이 좋습니다.
- return "1. 홈 화면에서 아무 곳이나 길게 눌러주세요." + return NSLocalizedString("widget_guide_step1", comment: "홈 화면에서 아무 곳이나 길게 눌러주세요.")비슷하게 다른 텍스트들도 처리하세요.
121-121: 애니메이션 파일 참조 방식 개선 필요애니메이션 파일 이름이 하드코딩되어 있습니다. 상수로 분리하여 관리하는 것이 코드 유지 보수성을 높일 수 있습니다.
- let animation = LottieAnimation.named("add_home_screen_widget", bundle: DesignSystemResources.bundle) + let animation = LottieAnimation.named(WidgetAnimations.addHomeScreen, bundle: DesignSystemResources.bundle)그리고 상수 정의:
enum WidgetAnimations { static let addHomeScreen = "add_home_screen_widget" }
139-148: 타이머 최적화 고려현재 타이머 구현은 0.5초마다 업데이트됩니다. 이는 배터리 소모를 증가시킬 수 있습니다. 가이드 텍스트가 4초, 2.5초 및 그 이후로 변경되므로, 타이머 간격을 1초로 설정하고 상태 변경이 필요한 시점에 맞추는 것이 더 효율적일 수 있습니다.
- every: 0.5, + every: 1.0,Projects/Feature/AddWidgetFeature/Sources/WidgetReperesentation.swift (1)
3-3: 구조체 이름에 오타가 있습니다."WidgetReperesentation"에 오타가 있습니다. "WidgetRepresentation"으로 수정하는 것이 좋습니다.
-public struct WidgetReperesentation: Sendable, Equatable, Hashable { +public struct WidgetRepresentation: Sendable, Equatable, Hashable {Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift (1)
27-45: onAppear 액션 처리가 효율적으로 구현되었습니다.onAppear 시 위젯 목록을 생성하고 loadWidgets 액션을 보내는 방식이 효율적입니다. 하지만 하드코딩된 위젯 목록은 별도의 상수나 팩토리 메서드로 분리하는 것이 유지보수에 더 용이할 수 있습니다.
case .onAppear: - let widgets: [WidgetReperesentation] = [ - WidgetReperesentation(kind: .mealAndTimetable, family: .systemMedium), - WidgetReperesentation(kind: .meal, family: .systemSmall), - WidgetReperesentation(kind: .meal, family: .systemMedium), - WidgetReperesentation(kind: .meal, family: .systemLarge), - WidgetReperesentation(kind: .meal, family: .accessory), - WidgetReperesentation(kind: .meal, family: .controlCenter), - WidgetReperesentation(kind: .timetable, family: .systemSmall), - WidgetReperesentation(kind: .timetable, family: .systemMedium), - WidgetReperesentation(kind: .timetable, family: .systemLarge), - WidgetReperesentation(kind: .timetable, family: .controlCenter) - ] + let widgets = createAvailableWidgets() return .send(.loadWidgets(widgets))새로운 private 메서드를 추가:
private func createAvailableWidgets() -> [WidgetReperesentation] { return [ WidgetReperesentation(kind: .mealAndTimetable, family: .systemMedium), WidgetReperesentation(kind: .meal, family: .systemSmall), WidgetReperesentation(kind: .meal, family: .systemMedium), WidgetReperesentation(kind: .meal, family: .systemLarge), WidgetReperesentation(kind: .meal, family: .accessory), WidgetReperesentation(kind: .meal, family: .controlCenter), WidgetReperesentation(kind: .timetable, family: .systemSmall), WidgetReperesentation(kind: .timetable, family: .systemMedium), WidgetReperesentation(kind: .timetable, family: .systemLarge), WidgetReperesentation(kind: .timetable, family: .controlCenter) ] }Projects/Feature/AddWidgetFeature/Sources/Components/TimetableView.swift (2)
5-47: 위젯 패밀리에 따른 적절한 뷰 분기 처리가 잘 구현되었습니다.다양한 위젯 크기와 유형에 따라 적절한 뷰를 표시하는 분기 처리가 잘 구현되어 있습니다. 하지만
default케이스에서EmptyView()를 반환하는 것보다는 모든 케이스를 명시적으로 처리하거나, 지원하지 않는 케이스에 대한 메시지를 보여주는 것이 더 나을 수 있습니다.- default: - EmptyView() + case .systemExtraLarge, .accessory, .accessoryRectangular, .accessoryCircular: + Text("지원하지 않는 위젯 크기입니다.") + .frame(maxWidth: .infinity, maxHeight: .infinity) + .background(Color.backgroundMain) + .cornerRadius(12)
56-56: 시간표 항목이 여러 뷰에서 중복 정의되어 있습니다.시간표 항목 배열이 여러 뷰에서 중복으로 정의되어 있습니다. 이를 상수로 분리하여 재사용하면 유지보수가 더 용이해질 것입니다.
클래스 상단에 다음과 같은 상수를 정의하고, 각 뷰에서 참조하도록 수정할 수 있습니다:
private let sampleTimetableItems = ["현대문학 감상", "음악 감상과 비평", "운동과 건강", "영미 문학 읽기", "생활과 과학", "미적분", "논술"]그런 다음 각 뷰에서 다음과 같이 사용:
-let items = ["현대문학 감상", "음악 감상과 비평", "운동과 건강", "영미 문학 읽기", "생활과 과학", "미적분", "논술"] +let items = sampleTimetableItemsAlso applies to: 99-99, 149-149
Projects/Feature/AddWidgetFeature/Sources/Components/MealView.swift (5)
69-69: 식단 아이템 강조 로직 분리 필요"구운 채소" 항목에 특별한 색상을 적용하는 로직이 하드코딩되어 있습니다. 이는 유지보수 측면에서 좋지 않습니다. 강조할 항목을 결정하는 로직을 별도로 분리하는 것이 좋습니다.
- .foregroundColor(meal == "구운 채소" ? .point : .extraBlack) + .foregroundColor(shouldHighlight(meal) ? .point : .extraBlack)
80-145: MealMediumView 코드 최적화 필요
MealMediumView내 LazyHGrid 구현 부분에서 GeometryReader를 사용하여 크기를 계산하고 있는데, 이 부분이 복잡하고 성능에 영향을 줄 수 있습니다. 가능하다면 더 단순하고 선언적인 방식으로 레이아웃을 구성하는 것이 좋습니다.
216-228: 접근성 고려 필요텍스트가 쉼표로 구분된 긴 문자열로 표시되고 있어 접근성 측면에서 좋지 않을 수 있습니다. VoiceOver와 같은 접근성 기능을 위한 적절한 레이블과 속성을 추가하는 것이 좋습니다.
Text("아침 - \(items.joined(separator: ","))") .twFont(.caption1) .lineLimit(nil) + .accessibilityLabel("아침 식단: \(items.joined(separator: ", "))")
1-297: 전반적인 국제화(i18n) 지원 부족코드 전체에 걸쳐 한국어 텍스트가 하드코딩되어 있습니다. 다국어 지원을 위해 문자열을 현지화(localization)하는 것이 좋습니다.
- Text("[아침]") + Text(NSLocalizedString("meal_breakfast_title", comment: "아침 식단 타이틀"))
4-52: 접근 제어 수준 개선 필요
MealView구조체가public또는internal접근 수준으로 명시되어 있지 않습니다. 이 컴포넌트가 다른 모듈에서 사용될 예정이라면 명시적인 접근 제어 수준을 추가하는 것이 좋습니다.- struct MealView: View { + public struct MealView: View {
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (4)
Projects/UserInterface/DesignSystem/Resources/Images.xcassets/Images/CircularMeal.imageset/meal-icon 1.svgis excluded by!**/*.svgProjects/UserInterface/DesignSystem/Resources/Images.xcassets/Images/CircularMeal.imageset/meal-icon 2.svgis excluded by!**/*.svgProjects/UserInterface/DesignSystem/Resources/Images.xcassets/Images/CircularMeal.imageset/meal-icon.svgis excluded by!**/*.svgProjects/UserInterface/DesignSystem/Resources/Images.xcassets/Images/WidgetSetting.imageset/widget-setting.svgis excluded by!**/*.svg
📒 Files selected for processing (27)
Package.resolved(1 hunks)Package.swift(1 hunks)Plugin/DependencyPlugin/ProjectDescriptionHelpers/Dependency+SPM.swift(1 hunks)Plugin/DependencyPlugin/ProjectDescriptionHelpers/ModulePaths.swift(1 hunks)Projects/App/iOS/Sources/Application/AppDelegate.swift(1 hunks)Projects/App/iOS/Support/Info.plist(2 hunks)Projects/Feature/AddWidgetFeature/Project.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/AddWidgetView.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/Components/MealAndTimetableView.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/Components/MealView.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/Components/TimetableView.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/Constants/MealPartTime.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/Constants/WidgetSizeConstants.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/WidgetReperesentation.swift(1 hunks)Projects/Feature/BaseFeature/Project.swift(1 hunks)Projects/Feature/SettingsFeature/Project.swift(1 hunks)Projects/Feature/SettingsFeature/Sources/SettingsCore.swift(6 hunks)Projects/Feature/SettingsFeature/Sources/SettingsView.swift(4 hunks)Projects/Shared/ComposableArchitectureWrapper/Sources/Sources.swift(0 hunks)Projects/Shared/ComposableArchitectureWrapper/Sources/export.swift(1 hunks)Projects/Shared/TWLog/Sources/TWUserProperty.swift(1 hunks)Projects/UserInterface/DesignSystem/Resources/Images.xcassets/Images/CircularMeal.imageset/Contents.json(1 hunks)Projects/UserInterface/DesignSystem/Resources/Images.xcassets/Images/WidgetSetting.imageset/Contents.json(1 hunks)Projects/UserInterface/DesignSystem/Sources/TWBottomSheet/TWBottomSheet.swift(4 hunks)Projects/UserInterface/DesignSystem/Sources/TWImage/TWImage.swift(1 hunks)Tuist/ProjectDescriptionHelpers/GenerateEnvironment.swift(1 hunks)
💤 Files with no reviewable changes (1)
- Projects/Shared/ComposableArchitectureWrapper/Sources/Sources.swift
✅ Files skipped from review due to trivial changes (5)
- Projects/Shared/ComposableArchitectureWrapper/Sources/export.swift
- Projects/UserInterface/DesignSystem/Resources/Images.xcassets/Images/WidgetSetting.imageset/Contents.json
- Tuist/ProjectDescriptionHelpers/GenerateEnvironment.swift
- Projects/UserInterface/DesignSystem/Resources/Images.xcassets/Images/CircularMeal.imageset/Contents.json
- Projects/Feature/AddWidgetFeature/Sources/Constants/WidgetSizeConstants.swift
🧰 Additional context used
🪛 SwiftLint (0.57.0)
Projects/Feature/AddWidgetFeature/Sources/AddWidgetView.swift
[Warning] 7-7: swiftlint command does not specify any rules
(invalid_swiftlint_command)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: ⚙️ Prepare for CI
🔇 Additional comments (33)
Projects/Shared/TWLog/Sources/TWUserProperty.swift (1)
11-11: 사용자 속성에 activeWatch 추가 확인.위젯 기능 확장의 일환으로
activeWatch사용자 속성이 적절하게 추가되었습니다. 이는 앱이 워치 앱 설치 여부를 추적할 수 있게 해주는 유용한 기능입니다.Plugin/DependencyPlugin/ProjectDescriptionHelpers/Dependency+SPM.swift (1)
17-17: Lottie 라이브러리 의존성 추가 확인.Lottie 라이브러리가 SPM 의존성으로 적절하게 추가되었습니다. 이 라이브러리는 애니메이션 구현에 유용하며, 위젯 기능 확장에 도움이 될 것입니다.
Projects/Feature/SettingsFeature/Project.swift (1)
14-14: AddWidgetFeature 모듈 의존성 추가 확인.SettingsFeature 모듈에 AddWidgetFeature 모듈 의존성이 적절하게 추가되었습니다. 이는 설정 화면에서 위젯 관련 기능을 통합하는 데 필요한 변경사항입니다.
Plugin/DependencyPlugin/ProjectDescriptionHelpers/ModulePaths.swift (1)
27-27: ModulePaths에 AddWidgetFeature 추가 확인.Feature 열거형에 AddWidgetFeature 케이스가 적절하게 추가되었습니다. 위젯 기능 모듈을 프로젝트 구조에 통합하기 위한 필수적인 변경사항입니다.
Projects/Feature/AddWidgetFeature/Sources/Constants/MealPartTime.swift (1)
1-18: 코드가 잘 구성되어 있습니다!식사 시간을 표현하는 열거형이 명확하게 정의되어 있으며, 각 케이스에 대한 표시 문자열도 적절히 구현되었습니다.
Int및CaseIterable프로토콜 채택은 향후 확장성과 사용성을 위해 좋은 선택입니다.Package.swift (1)
30-31: Lottie 라이브러리 추가가 적절히 이루어졌습니다.Airbnb의 Lottie 라이브러리를 추가하여 애니메이션 기능을 지원할 수 있게 되었습니다. 위젯 기능 개선에 유용하게 활용될 것으로 보입니다.
Projects/Feature/BaseFeature/Project.swift (1)
20-21: BaseFeature에 Lottie 의존성 추가가 적절합니다.Package.swift에 추가된 Lottie 라이브러리를 BaseFeature 모듈에서 사용할 수 있도록 의존성이 올바르게 설정되었습니다.
Projects/Feature/AddWidgetFeature/Project.swift (1)
1-13: AddWidgetFeature 모듈 구성이 적절합니다.새로운 위젯 기능을 위한 프로젝트 설정이 프로젝트 구조 규칙에 맞게 잘 구성되었습니다. BaseFeature와 ComposableArchitectureWrapper에 대한 의존성 설정이 적절하여 위젯 기능을 원활하게 구현할 수 있을 것입니다.
Projects/App/iOS/Sources/Application/AppDelegate.swift (1)
42-42: WCSession 설치 여부에 대한 사용자 속성 추가가 잘 구현되었습니다.Watch 앱 설치 여부를 사용자 속성으로 추가하는 것은 분석 데이터를 풍부하게 만드는 좋은 방법입니다. 이렇게 하면 사용자 행동과 앱 사용 패턴에 대한 더 많은 통찰력을 얻을 수 있습니다.
Projects/App/iOS/Support/Info.plist (2)
22-22: 버전 업데이트가 적절히 이루어졌습니다.버전이 7.1에서 7.2로 업데이트되었습니다. 위젯 추가 설정과 같은 새로운 기능에 맞게 버전 번호가 적절히 증가되었습니다.
35-35: 빌드 번호가 적절히 증가되었습니다.빌드 번호가 60에서 63으로 증가되었습니다. 이는 새로운 기능 추가에 따른 적절한 빌드 번호 증가입니다.
Projects/UserInterface/DesignSystem/Sources/TWImage/TWImage.swift (1)
19-20: 새 이미지 자산이 적절하게 추가되었습니다.위젯 관련 기능을 위한 새 이미지 자산
circularMeal과widgetSetting이 추가되었습니다. 이는 PR의 위젯 추가 설정 기능과 잘 연결됩니다.Package.resolved (1)
120-128:Details
✅ Verification successful
Lottie 애니메이션 라이브러리가 추가되었습니다.
위젯 설정 UI를 위한 애니메이션 기능을 제공하기 위해 Lottie 라이브러리가 추가되었습니다. Lottie는 고품질 애니메이션을 쉽게 구현할 수 있게 해주는 좋은 선택입니다.
Lottie 라이브러리가 프로젝트에서 올바르게 사용되고 있는지, 그리고 이 버전이 최신 안정 버전인지 확인해보세요:
🌐 Web query:
What is the latest stable version of Lottie for iOS?💡 Result:
The latest stable version of Lottie for iOS is 4.5.1, as published on npmjs in January 2025[4]. This version includes recent updates such as:
- Compatibility requirements:
- Xcode 15/Swift 5.9 minimum[1][3]
- iOS 13.0/macOS 10.15 minimum deployment targets[1]
Key updates leading to this version include:
- Privacy manifest support for App Store compliance[1][3]
- VisionOS platform compatibility[1][3]
- SwiftUI component integration (
LottieView,LottieSwitch,LottieButton)[1][7]- Core Animation rendering engine improvements with time remapping[1][3]
For React Native developers, the latest Lottie React Native v7.1.0 uses Lottie-iOS 4.4.0 under the hood[2], though the native iOS library itself has newer standalone releases.
Citations:
- 1: https://github.com/airbnb/lottie-ios/releases
- 2: https://github.com/lottie-react-native/lottie-react-native/releases
- 3: https://github.com/airbnb/lottie-spm/releases
- 4: https://www.npmjs.com/package/lottie-ios
- 5: https://cocoapods.org/pods/lottie-ios-bb
- 6: https://www.npmjs.com/package/lottie-react-native
- 7: https://airbnb.io/projects/lottie-ios/
- 8: https://swift.libhunt.com/lottie-ios-changelog
- 9: https://android.googlesource.com/platform/external/lottie/+/74746cd7798dccc432cd84b2ee8f1c741031a0d7/README.md
- 10: lottie-react-native/lottie-react-native#1216
확인 완료: Lottie 라이브러리 버전 검증됨
Package.resolved파일에 명시된"version" : "4.5.1"값이 최신 안정 버전임이 웹 검색 결과를 통해 확인되었습니다.- 4.5.1 버전은 January 2025 기준 최신 업데이트가 반영된 안정된 릴리즈입니다.
- 현재 Lottie 라이브러리 추가 및 관련 설정은 적절하게 이루어졌으므로 수정할 사항은 없습니다.
Projects/UserInterface/DesignSystem/Sources/TWBottomSheet/TWBottomSheet.swift (1)
10-10: 배경색 커스터마이징 지원 추가 👍바텀 시트의 배경색을 커스터마이징할 수 있도록 기능이 추가되었습니다. 이는 위젯 가이드와 같은 다양한 컨텍스트에서 바텀 시트의 유연성을 향상시킵니다.
Also applies to: 34-34, 39-39, 60-60, 90-97
Projects/Feature/AddWidgetFeature/Sources/Components/MealAndTimetableView.swift (1)
4-17: 위젯 패밀리 타입에 따른 분기 처리가 필요합니다.현재
.systemMedium케이스만 구현되어 있고, 다른 케이스들은EmptyView()로 처리됩니다. 다른 위젯 크기도 지원할 계획이 있다면 해당 뷰도 구현하거나, 아니라면 주석으로 미구현 이유를 명시하는 것이 좋습니다.다른 위젯 사이즈(small, large)에 대한 지원 계획이 있으신가요?
Projects/Feature/AddWidgetFeature/Sources/AddWidgetView.swift (1)
25-31: 바텀 시트 배경색 선택이 적절합니다.위젯 가이드를 표시하는 바텀 시트의 배경색을
.cardBackground로 설정한 것은 UI 일관성 측면에서 좋은 선택입니다.Projects/Feature/SettingsFeature/Sources/SettingsCore.swift (2)
1-1: 위젯 설정 기능 통합이 잘 구현되었습니다.위젯 설정 기능이 기존 설정 기능에 잘 통합되었습니다. AddWidgetFeature 모듈을 가져오고, 상태와 액션을 정의하며, reducer에서 처리하는 방식이 기존 코드 패턴을 잘 따르고 있습니다.
Also applies to: 28-28, 42-42, 51-51, 120-122
215-215: 서브 피처 등록이 올바르게 되었습니다.AddWidgetCore가 서브 피처로 올바르게 등록되었습니다. 이렇게 하면 설정 화면에서 위젯 관리 기능으로 스무스하게 전환될 수 있습니다.
Projects/Feature/SettingsFeature/Sources/SettingsView.swift (6)
1-1: AddWidgetFeature 임포트가 적절히 추가되었습니다.위젯 기능을 설정 화면에 통합하기 위한 임포트가 올바르게 추가되었습니다.
28-29: 위젯 설정 기능이 적절히 추가되었습니다.VStack에 위젯 설정 기능이 추가되었고, 기존 알레르기 설정 전에 위치하도록 잘 구성되었습니다.
89-95: 위젯 추가 설정 UI가 일관된 스타일로 구현되었습니다.기존 설정 옵션들과 동일한 패턴으로 위젯 추가 설정 UI가 잘 구현되었습니다. 상단 모서리가 둥글게 처리되어 첫 번째 아이템으로서의 시각적 일관성도 유지됩니다.
91-91: 액션 연결이 적절히 구현되었습니다.탭 이벤트가 올바르게
addWidgetButtonDidTap액션으로 전달됩니다.
93-93: 위젯 설정 아이콘과 텍스트가 명확합니다.아이콘
.widgetSetting과 텍스트 "위젯 추가"를 사용하여 사용자에게 기능을 명확하게 전달합니다.
238-245: 위젯 추가 화면으로의 네비게이션이 적절히 구현되었습니다.
NavigationLinkStore를 사용하여 위젯 추가 화면으로의 네비게이션이 다른 설정 화면과 일관된 방식으로 구현되었습니다.Projects/Feature/AddWidgetFeature/Sources/WidgetReperesentation.swift (2)
4-16: 위젯 종류 열거형이 명확하게 정의되었습니다.위젯 종류에 대한 열거형이 잘 정의되어 있으며, 각 케이스에 대한 제목도 적절하게 제공됩니다.
18-40: 위젯 크기 타입 열거형이 다양한 케이스를 포함합니다.다양한 위젯 크기 및 위치(홈화면, 잠금화면, 제어센터)에 대한 케이스가 잘 정의되어 있습니다.
Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift (3)
4-15: Reducer 구조가 TCA 패턴을 잘 따르고 있습니다.상태(State)가 명확하게 정의되어 있으며, 필요한 프로퍼티들이 포함되어 있습니다.
17-23: 액션 정의가 명확합니다.위젯 선택, 로드, 가이드 표시, 완료 등 필요한 액션이 잘 정의되어 있습니다.
54-58: 위젯 가이드 표시 로직이 간결합니다.가이드가 표시되지 않을 때 선택된 위젯을 초기화하는 로직이 간결하게 구현되었습니다.
Projects/Feature/AddWidgetFeature/Sources/Components/TimetableView.swift (1)
178-230: 제어 센터용 위젯 뷰가 시각적으로 일관된 디자인으로 구현되었습니다.제어 센터용 소형, 중형, 대형 위젯이 iOS의 디자인 가이드라인에 맞게 일관되게 구현되었습니다. Material 배경과 아이콘, 텍스트 배치가 적절합니다.
Projects/Feature/AddWidgetFeature/Sources/Components/MealView.swift (3)
4-52: 위젯 패밀리 타입에 따른 뷰 전환 구현위젯 패밀리 타입에 따라 다양한 뷰를 표시하는 접근 방식이 잘 구성되어 있습니다. 다만,
default케이스에서EmptyView()를 반환하는 것이 안전하지만, 실제로 이 케이스가 발생할 가능성이 있는지 검토해보세요.
94-115: iOS 17 이전 버전 대응 필요iOS 17 이상에서만 작동하는 코드 블록이 있으나, 이전 버전에 대한 대체 UI가 제공되지 않습니다. 앱의 최소 지원 버전이 iOS 17 미만이라면 대체 UI를 구현해야 합니다.
if #available(iOS 17.0, *) { ForEach([MealPartTime.breakfast, .lunch, .dinner], id: \.self) { partTime in // 현재 코드 } +} else { + // iOS 17 미만 버전을 위한 대체 UI + Text(MealPartTime.breakfast.display) + .twFont(.body3, color: .extraWhite) + .padding(.horizontal, 12) + .background( + RoundedRectangle(cornerRadius: 8) + .fill(Color.extraBlack) + .frame(height: 24) + ) }
4-297: 접근성 및 다크모드 지원 검토 필요현재 구현은 색상 사용에 있어 다크모드 지원이 고려되지 않은 것으로 보입니다. 또한, 접근성 지원을 위한 Dynamic Type 대응이 부족합니다. 특히 폰트 크기 조절과 관련하여
minimumScaleFactor를 사용하는 것보다 Dynamic Type을 제대로 지원하는 것이 좋습니다.
Projects/Feature/AddWidgetFeature/Sources/Components/MealAndTimetableView.swift
Show resolved
Hide resolved
Projects/Feature/AddWidgetFeature/Sources/Components/MealAndTimetableView.swift
Show resolved
Hide resolved
| private struct MealLargeView: View { | ||
| var body: some View { | ||
| RoundedRectangle(cornerRadius: 16) | ||
| .fill(Color.backgroundMain) | ||
| .overlay { | ||
| VStack(spacing: 4) { | ||
| HStack(spacing: 4) { | ||
| Text("[아침]") | ||
| .twFont(.body3, color: .extraBlack) | ||
|
|
||
| Spacer() | ||
|
|
||
| HStack(spacing: 0) { | ||
| if #available(iOS 17.0, *) { | ||
| ForEach([MealPartTime.breakfast, .lunch, .dinner], id: \.self) { partTime in | ||
| Button {} label: { | ||
| let isSelected = partTime == .breakfast | ||
| Text(partTime.display) | ||
| .twFont(.body3, color: isSelected ? .extraWhite : .textSecondary) | ||
| .minimumScaleFactor(0.5) | ||
| .padding(.horizontal, 12) | ||
| .background { | ||
| if isSelected { | ||
| RoundedRectangle(cornerRadius: 8) | ||
| .fill(Color.extraBlack) | ||
| .frame(height: 24) | ||
| } else { | ||
| Color.clear | ||
| } | ||
| } | ||
| } | ||
| .buttonStyle(.borderless) | ||
| .frame(height: 24) | ||
| } | ||
| } | ||
| } | ||
| } | ||
|
|
||
| let items = ["퀴노아밥", "테리야키 그릴스테이크", "한우 불고기", "팝스터 쉐이크", "구운 채소", "가지 라따뚜이", "깍두기", "자몽에이드"] | ||
| VStack(alignment: .leading, spacing: 0) { | ||
| ForEach(items, id: \.self) { meal in | ||
| HStack { | ||
| Text(meal) | ||
| .twFont(.body1) | ||
| .frame(maxHeight: .infinity) | ||
| .foregroundColor(meal == "구운 채소" ? .point : .extraBlack) | ||
|
|
||
| Spacer() | ||
| } | ||
| .padding(.horizontal, 8) | ||
| } | ||
| } | ||
| .padding(.top, 4) | ||
| .frame(maxWidth: .infinity, maxHeight: .infinity) | ||
| .background { | ||
| Color.cardBackground | ||
| } | ||
| .cornerRadius(8) | ||
|
|
||
| Text("\(String(format: "%.1f", 1290)) kcal") | ||
| .twFont(.caption1, color: .textSecondary) | ||
| .frame(maxWidth: .infinity, alignment: .trailing) | ||
| } | ||
| .padding(8) | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
🛠️ Refactor suggestion
MealLargeView와 MealMediumView 간 중복 코드
MealLargeView와 MealMediumView에 중복되는 코드가 많습니다. 특히 식사 시간대 선택 버튼 UI 부분이 거의 동일합니다. 이 부분을 별도의 컴포넌트로 추출하여 재사용하면 코드의 유지보수성이 향상될 것입니다.
// 추출할 공통 컴포넌트 예시
private struct MealTimeSelector: View {
var selectedPartTime: MealPartTime
var onSelect: (MealPartTime) -> Void
var body: some View {
HStack(spacing: 0) {
if #available(iOS 17.0, *) {
ForEach([MealPartTime.breakfast, .lunch, .dinner], id: \.self) { partTime in
Button {
onSelect(partTime)
} label: {
let isSelected = partTime == selectedPartTime
Text(partTime.display)
.twFont(.body3, color: isSelected ? .extraWhite : .textSecondary)
.minimumScaleFactor(0.5)
.padding(.horizontal, 12)
.background {
if isSelected {
RoundedRectangle(cornerRadius: 8)
.fill(Color.extraBlack)
.frame(height: 24)
} else {
Color.clear
}
}
}
.buttonStyle(.borderless)
.frame(height: 24)
}
}
}
}
}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift (3)
5-16: 타입 이름에 오타가 있습니다.
WidgetReperesentation에 오타가 있는 것 같습니다. 올바른 철자는WidgetRepresentation입니다(누락된 'r'). 이 오타는 전체 코드베이스에서 일관되게 사용해야 합니다.- public var selectedWidget: WidgetReperesentation? - public var availableWidgets: [WidgetReperesentation] + public var selectedWidget: WidgetRepresentation? + public var availableWidgets: [WidgetRepresentation]- public init( - availableWidgets: [WidgetReperesentation] = [] + public init( + availableWidgets: [WidgetRepresentation] = []
28-46: 위젯 종류가 하드코딩되어 있습니다.위젯 목록이 하드코딩되어 있어 확장성에 제한이 있을 수 있습니다. 이 배열을 별도의 상수나 설정 파일로 분리하는 것이 코드 관리 및 확장성 측면에서 더 좋을 것 같습니다.
- let widgets: [WidgetReperesentation] = [ - WidgetReperesentation(kind: .mealAndTimetable, family: .systemMedium), - WidgetReperesentation(kind: .meal, family: .systemSmall), - WidgetReperesentation(kind: .meal, family: .systemMedium), - WidgetReperesentation(kind: .meal, family: .systemLarge), - WidgetReperesentation(kind: .meal, family: .accessory), - WidgetReperesentation(kind: .meal, family: .controlCenter), - WidgetReperesentation(kind: .timetable, family: .systemSmall), - WidgetReperesentation(kind: .timetable, family: .systemMedium), - WidgetReperesentation(kind: .timetable, family: .systemLarge), - WidgetReperesentation(kind: .timetable, family: .controlCenter) - ] + return .send(.loadWidgets(WidgetDefaults.availableWidgets))이를 위해 새로운 파일이나 extension에 다음과 같은 코드를 추가할 수 있습니다:
// WidgetDefaults.swift 또는 AddWidgetCore+Extensions.swift public enum WidgetDefaults { public static let availableWidgets: [WidgetRepresentation] = [ WidgetRepresentation(kind: .mealAndTimetable, family: .systemMedium), WidgetRepresentation(kind: .meal, family: .systemSmall), // 나머지 위젯들... ] }
1-67:WidgetReperesentation과 관련된 추가 정보가 필요합니다.이 파일에서는
WidgetReperesentation이 어떻게 정의되어 있는지 알 수 없습니다. 코드에서 사용된.mealAndTimetable,.meal,.timetable과 같은kind와.systemMedium,.systemSmall등의family값에 대한 문서화가 필요합니다. 특히.mealAndTimetable은.systemMedium에만 정의되어 있는데, 이에 대한 설명이 필요합니다.도움이 필요하시면
WidgetReperesentation타입과 관련 열거형에 대한 문서화나 구현을 제안해 드릴 수 있습니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: ⚙️ Prepare for CI
🔇 Additional comments (6)
Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift (6)
1-4: Import 구문이 명확하게 정의되어 있습니다.ComposableArchitecture 패턴을 사용하고 있으며, Intents 프레임워크를 통해 위젯 기능을 구현하기 위한 적절한 import 구문을 사용하고 있습니다.
18-24: Action 열거형이 잘 정의되어 있습니다.위젯 관리에 필요한 모든 액션이 명확하게 정의되어 있습니다. 사용자 상호작용부터 내부 상태 관리까지 필요한 모든 액션이 포함되어 있습니다.
26-26: initializer가 적절하게 구현되어 있습니다.비어있는 initializer는 이 경우 적합합니다. 초기 설정이 필요 없는 Redux 스타일의 Reducer에 적합합니다.
47-53: 상태 업데이트 로직이 적절하게 구현되어 있습니다.위젯 로드 및 선택 액션에 대한 상태 업데이트 로직이 간결하고 명확합니다. 부수 효과 없이 상태만 업데이트하는 순수 함수 접근 방식이 잘 지켜지고 있습니다.
55-63: 상태 초기화 로직이 명확합니다.가이드가 표시되지 않을 때와 위젯 추가가 완료되었을 때
selectedWidget을 초기화하는 로직이 명확하게 구현되어 있습니다. 사용자 경험을 고려한 적절한 상태 관리입니다.
32-43:Details
❓ Verification inconclusive
WidgetReperesentation타입에 대한 정의가 없습니다.
WidgetReperesentation타입과 그 내부에 사용된kind,family열거형에 대한 정의가 이 파일에 없습니다. 이 타입이 프로젝트 내 다른 곳에 정의되어 있는지 확인해야 합니다.
🏁 Script executed:
#!/bin/bash # WidgetReperesentation 타입 정의 찾기 fd -e swift | xargs grep -l "WidgetReperesentation" | grep -v "AddWidgetCore.swift" # kind, family 열거형 정의 찾기 fd -e swift | xargs grep -l "enum.*kind\|enum.*family" | grep -v "AddWidgetCore.swift"Length of output: 521
타입 정의 위치 및 열거형 확인 필요
WidgetReperesentation타입은Projects/Feature/AddWidgetFeature/Sources/WidgetReperesentation.swift파일에서 실제로 정의되어 있음을 확인했습니다.- 하지만
kind와family에 사용되는 열거형의 정의는 검색 결과로 명확하게 드러나지 않았습니다.
→ 해당 열거형들이WidgetReperesentation.swift내부에 포함되어 있거나, 별도의 파일에 정의되어 있는지 추가 확인이 필요합니다.
baekteun
force-pushed
the
190-add-widget-setting
branch
from
a72726b to
a8b2c10
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift (3)
5-16: 'WidgetReperesentation'의 철자가 잘못된 것 같습니다.
WidgetReperesentation은 철자가 잘못된 것으로 보입니다. 올바른 철자는WidgetRepresentation(r 누락)일 것입니다.다음과 같이 수정하는 것이 좋겠습니다:
- public var selectedWidget: WidgetReperesentation? - public var availableWidgets: [WidgetReperesentation] + public var selectedWidget: WidgetRepresentation? + public var availableWidgets: [WidgetRepresentation]그리고 초기화 메서드에도 동일하게 적용해야 합니다:
- public init( - availableWidgets: [WidgetReperesentation] = [] - ) { + public init( + availableWidgets: [WidgetRepresentation] = [] + ) {
28-46: 하드코딩된 위젯 목록을 분리하는 것이 좋겠습니다.
onAppear액션에서 위젯 목록이 하드코딩되어 있습니다. 이 부분을 별도의 함수나 상수로 분리하면 코드의 가독성과 유지보수성이 향상될 것입니다.다음과 같이 분리하는 것을 제안합니다:
public var body: some ReducerOf<AddWidgetCore> { Reduce { state, action in switch action { case .onAppear: - let widgets: [WidgetReperesentation] = [ - WidgetReperesentation(kind: .mealAndTimetable, family: .systemMedium), - WidgetReperesentation(kind: .meal, family: .systemSmall), - WidgetReperesentation(kind: .meal, family: .systemMedium), - WidgetReperesentation(kind: .meal, family: .systemLarge), - WidgetReperesentation(kind: .meal, family: .accessory), - WidgetReperesentation(kind: .meal, family: .controlCenter), - WidgetReperesentation(kind: .timetable, family: .systemSmall), - WidgetReperesentation(kind: .timetable, family: .systemMedium), - WidgetReperesentation(kind: .timetable, family: .systemLarge), - WidgetReperesentation(kind: .timetable, family: .controlCenter) - ] + return .send(.loadWidgets(Self.defaultWidgets))그리고 클래스 내부에 static 속성을 추가합니다:
private static let defaultWidgets: [WidgetRepresentation] = [ WidgetRepresentation(kind: .mealAndTimetable, family: .systemMedium), WidgetRepresentation(kind: .meal, family: .systemSmall), WidgetRepresentation(kind: .meal, family: .systemMedium), WidgetRepresentation(kind: .meal, family: .systemLarge), WidgetRepresentation(kind: .meal, family: .accessory), WidgetRepresentation(kind: .meal, family: .controlCenter), WidgetRepresentation(kind: .timetable, family: .systemSmall), WidgetRepresentation(kind: .timetable, family: .systemMedium), WidgetRepresentation(kind: .timetable, family: .systemLarge), WidgetRepresentation(kind: .timetable, family: .controlCenter) ]
55-60: showWidgetGuide 로직에 주석 추가가 필요합니다.
showWidgetGuide액션에서isShowing이false일 때만selectedWidget을nil로 설정하는 로직의 의도가 명확하지 않습니다. 이 부분에 주석을 추가하여 의도를 명확히 설명하는 것이 좋겠습니다.다음과 같이 주석을 추가하는 것을 제안합니다:
case let .showWidgetGuide(isShowing): + // 위젯 가이드가 닫힐 때 선택된 위젯을 초기화합니다. + // 가이드가 열릴 때는 선택 상태를 유지하여 사용자 경험을 개선합니다. if isShowing == false { state.selectedWidget = nil } return .none
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
Projects/App/iOS/Sources/Application/AppDelegate.swift(1 hunks)Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- Projects/App/iOS/Sources/Application/AppDelegate.swift
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: ⚙️ Prepare for CI
🔇 Additional comments (6)
Projects/Feature/AddWidgetFeature/Sources/AddWidgetCore.swift (6)
1-4: 필요한 모듈들을 적절히 가져왔습니다.ComposableArchitecture를 활용한 구현에 필요한 라이브러리를 적절히 가져왔습니다.
18-24: 액션 정의가 명확합니다.위젯 관리에 필요한 액션들이 적절히 정의되어 있습니다. 각 액션의 역할이 명확하게 구분되어 있어 좋습니다.
47-50: loadWidgets 액션 처리가 적절합니다.위젯 목록을 상태에 업데이트하는 로직이 명확하게 구현되어 있습니다.
51-54: selectWidget 액션 처리가 적절합니다.선택된 위젯을 상태에 업데이트하는 로직이 간결하게 구현되어 있습니다.
61-64: addWidgetComplete 액션 처리가 적절합니다.위젯 추가 완료 시 선택된 위젯을 초기화하는 로직이 적절합니다.
1-67:Details
✅ Verification successful
WidgetRepresentation 타입 문서화가 필요합니다.
코드 전체적으로
WidgetReperesentation타입이 사용되고 있지만, 이 타입에 대한 정의나 문서화가 없습니다. 이 타입의 정의와 역할에 대한 문서화를 추가하는 것이 좋겠습니다.파일 상단에 다음과 같은 문서 주석을 추가하는 것을 고려해보세요:
/// AddWidgetCore는 위젯 추가 기능을 관리하는 리듀서입니다. /// 이 모듈은 `WidgetRepresentation` 타입을 사용하여 위젯의 종류와 크기를 표현합니다. /// `WidgetRepresentation`은 [참조할 파일 경로]에 정의되어 있습니다.또한
WidgetRepresentation타입이 정의된 곳을 확인하기 위해 다음 명령을 실행하는 것이 좋겠습니다:
🏁 Script executed:
#!/bin/bash # WidgetRepresentation 또는 WidgetReperesentation 타입의 정의를 찾습니다. fd -e swift | xargs grep -l "struct WidgetRep[er][er]esentation"Length of output: 136
위젯 타입 문서화 추가 필요
현재
AddWidgetCore.swift에서는WidgetReperesentation타입을 광범위하게 사용하고 있으나, 이 타입의 정의와 역할에 관한 문서가 부족합니다. 검색 결과, 해당 타입은
Projects/Feature/AddWidgetFeature/Sources/WidgetReperesentation.swift
파일에 정의되어 있음이 확인되었습니다.다음과 같이 파일 상단에 문서 주석을 추가하는 것을 권장합니다:
/// AddWidgetCore는 위젯 추가 기능을 관리하는 리듀서입니다. /// 이 모듈은 `WidgetReperesentation` 타입을 사용하여 위젯의 종류와 크기를 표현합니다. /// `WidgetReperesentation`은 Projects/Feature/AddWidgetFeature/Sources/WidgetReperesentation.swift에 정의되어 있습니다.이 주석을 추가하여 타입의 역할과 정의를 명확히 해주시면 좋겠습니다.
💡 개요
위젯 추가 설정 추가
Summary by CodeRabbit