feat: 입력 관련 컴포넌트 추가, 디자인 시스템 일부 수정#69
Merged
Merged
Conversation
## Walkthrough
이번 변경 사항에서는 네비게이션 바 설정, 체크박스, 라벨, 텍스트필드 등 UI 컴포넌트가 대거 추가 및 개선되었습니다. 또한, 여러 리소스 파일과 에셋, 공통 베이스 뷰/컨트롤러, 인셋 상수, 버튼 및 아이콘 시스템 등도 신규 도입 및 리팩터링되었습니다.
## Changes
| 파일/경로 요약 | 변경 요약 |
|---|---|
| `.github/workflows/ios_ci.yml`, `src/SupportingFiles/Scripts/tuist_module_init.sh` | CI 스크립트 실행 경로 및 모듈 초기화 스크립트에 디렉토리 이동 명령 추가 |
| `src/Projects/BKDesign/PreviewApp/Sources/Common/BaseView.swift`, `BaseViewController.swift` | 공통 베이스 뷰 및 베이스 뷰 컨트롤러 클래스 추가 |
| `src/Projects/BKDesign/PreviewApp/Sources/SceneDelegate.swift` | 네비게이션 컨트롤러 및 네비게이션 바 글로벌 스타일 설정 추가, 초기 진입 VC 변경 |
| `src/Projects/BKDesign/PreviewApp/Sources/View/CatalogViewController.swift`, `BKInputCatalogViewController.swift`, `BKInputCatalogView.swift` | 카탈로그 및 입력 컴포넌트 데모 뷰/컨트롤러 신규 추가 |
| `src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonTestViewController.swift`, `BKButtonGroupDemoViewController.swift` | 버튼 테스트/그룹 데모 VC에서 아이콘 처리 및 네비게이션 타이틀 설정 개선 |
| `src/Projects/BKDesign/Resources/Assets.xcassets/...` | 앱 아이콘, 체크박스, 아이콘 등 다양한 에셋 및 이미지셋 JSON 파일 신규 추가 |
| `src/Projects/BKDesign/Sources/Components/BKLabel.swift`, `BKDivider.swift`, `BKBaseTextField.swift`, `BKSearchTextField.swift`, `BKTextView.swift`, `BKCheckBox.swift` | 라벨, 디바이더, 텍스트필드, 서치필드, 텍스트뷰, 체크박스 등 UI 컴포넌트 클래스 신규 추가 |
| `src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKIcon.swift` → `BKImage` | 아이콘 및 체크박스 이미지 접근 방식 리팩터링 및 네임스페이스화 |
| `src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKInset.swift` | 인셋 상수 enum 신규 추가 |
| `src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift` | attributedString 생성 시 extraAttributes 파라미터 추가 |
| `src/Projects/BKDesign/Sources/Components/Button/BKButton.swift`, `src/Tuist/Package.swift` | 불필요한 빈 줄 및 주석 제거 등 사소한 코드 정리 |
| `src/Projects/Booket/Project.swift` | 리소스 glob 패턴 확장 및 주석 제거 |
| `src/Projects/Booket/Sources/SceneDelegate.swift` | 네비게이션 바 설정 메서드 추가 및 호출, 불필요한 괄호 제거 |
| `src/Projects/BKPresentation/Sources/AuthFlow/View/LoginView.swift` | 카카오 아이콘 이미지 할당 방식 간소화 |
## Sequence Diagram(s)
```mermaid
sequenceDiagram
actor User
participant SceneDelegate
participant UINavigationController
participant CatalogViewController
participant BKInputCatalogViewController
participant BKInputCatalogView
User->>SceneDelegate: 앱 실행
SceneDelegate->>UINavigationController: 생성 및 네비게이션 바 스타일 설정
SceneDelegate->>CatalogViewController: push (초기 진입)
User->>CatalogViewController: 버튼 선택
CatalogViewController->>BKInputCatalogViewController: push
BKInputCatalogViewController->>BKInputCatalogView: 뷰 로드 및 컴포넌트 표시Assessment against linked issues
Suggested labels
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 10
🧹 Nitpick comments (16)
.github/workflows/ios_ci.yml (1)
85-90: 스크립트 내부cd ../../와 워크플로 경로 설정이 중복됩니다.현재 스텝에서
working-directory: src/SupportingFiles/Scripts로 이동한 뒤, 스크립트가 다시cd ../../를 수행합니다. 두 위치가 서로 결합되어 의도한 대로src디렉터리에서 동작하긴 하지만, 어느 한쪽이 변경되면 즉시 깨질 수 있는 취약한 결합입니다.
- 스크립트 내부에서 절대 경로 계산(앞 코멘트 참고)을 도입하거나
- 워크플로의
working-directory를 다시src로 돌리고./SupportingFiles/Scripts/tuist_module_init.sh로 호출둘 중 하나로 정리해 의존성을 단일화하는 편이 유지보수에 유리합니다.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled.imageset/Contents.json (1)
1-23: 이미지 세트 메타데이터는 문제없으나 네이밍 규칙을 통일하는 편이 좋습니다.
checkbox-filled처럼 하이픈(-)으로 구분한 이름과, 다른 세트의checkbox-icononly_default-round처럼 언더스코어(_)-혼합 형식이 함께 존재합니다.
향후 검색‧자동화 스크립트 작성 시 혼재된 구분자는 유지보수 비용을 높일 수 있으므로 하나의 컨벤션(예: 모두 하이픈)으로 맞추는 것을 권장합니다.src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-round.imageset/Contents.json (1)
1-23:rendering-intent지정 여부 검토체크박스는 주로
template렌더링(틴트 컬러 적용)을 사용하지 않고 원본 색상을 유지합니다.
향후 다크 모드 등에서 색상 오염을 방지하려면 각 이미지 항목에"rendering-intent": "original"속성을 추가하는 것을 고려해 보세요.src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-round.imageset/Contents.json (1)
1-23: 네이밍 컨벤션 불일치다른 세트(
checkbox-disabled-round,checkbox-filled등)는 하이픈 기반인데 본 세트는 언더스코어와 하이픈이 혼합되어 있습니다.
일관성을 위해checkbox-icononly-default-round등 하나의 규칙으로 맞추는 것을 권장합니다.src/Projects/BKDesign/PreviewApp/Sources/View/CatalogViewController.swift (1)
7-23: 버튼 생성 로직의 중복을 제거하세요.세 개의 버튼이 동일한 패턴으로 생성되고 있습니다. 코드 중복을 줄이기 위해 헬퍼 함수를 만드는 것을 고려해보세요.
다음과 같이 리팩토링할 수 있습니다:
+private func makeButton(title: String) -> UIButton { + let button = UIButton(type: .system) + button.setTitle(title, for: .normal) + return button +} + -private let inputCatalogButton: UIButton = { - let button = UIButton(type: .system) - button.setTitle("BKInputCatalogViewController", for: .normal) - return button -}() +private lazy var inputCatalogButton = makeButton(title: "BKInputCatalogViewController")다른 버튼들도 동일하게 적용하면 코드가 더 간결해집니다.
src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKInset.swift (1)
5-11: enum 사용 방식은 좋으나 네이밍 일관성 개선 필요상수 네임스페이스로 enum을 사용하는 것은 좋은 선택입니다. 하지만 네이밍에 일관성이 부족합니다:
inset1,inset2,inset3,inset4는 순차적 네이밍inset20은 값 기반 네이밍의미적 네이밍을 고려해보세요:
public enum BKInset { - public static let inset1: CGFloat = 4 - public static let inset2: CGFloat = 13 - public static let inset3: CGFloat = 16 - public static let inset4: CGFloat = 44 - public static let inset20: CGFloat = 20 + public static let small: CGFloat = 4 + public static let medium: CGFloat = 16 + public static let large: CGFloat = 20 + public static let extraLarge: CGFloat = 44 + public static let custom13: CGFloat = 13 // 특별한 용도가 있다면 주석 추가 }src/Projects/BKDesign/PreviewApp/Sources/Common/BaseView.swift (1)
5-23: 템플릿 메서드 패턴 구현 우수, 접근 제어 검토 필요템플릿 메서드 패턴을 통한 구조화된 초기화는 좋은 접근입니다. 하지만 접근 제어를 고려해보세요:
-class BaseView: UIView { +open class BaseView: UIView { // MARK: - Initialize override init(frame: CGRect = .zero) { super.init(frame: frame) self.setupView() self.setupLayout() self.configure() } @available(*, unavailable) required init?(coder: NSCoder) { fatalError("init(coder:) has not been implemented") } // MARK: - Common Methods - func setupView() {} - func setupLayout() {} - func configure() {} + open func setupView() {} + open func setupLayout() {} + open func configure() {} }이렇게 하면 다른 모듈에서도 BaseView를 상속받아 사용할 수 있습니다.
src/Projects/Booket/Sources/SceneDelegate.swift (1)
60-90: 네비게이션 바 스타일링 구현 우수, 재사용성 고려 필요네비게이션 바 설정이 매우 포괄적이고 잘 구현되었습니다:
- 디자인 시스템 일관성: BKTextStyle, UIColor 확장 활용
- 커스텀 백 버튼: 적절한 아이콘과 정렬 설정
- 완전한 appearance 설정: 모든 네비게이션 바 상태 고려
하지만 유사한 설정이 다른 SceneDelegate에도 있을 수 있으므로 공통 유틸리티 고려를 제안합니다:
// BKDesign 모듈에 추가 고려 public struct BKNavigationBarConfigurator { public static func configure(_ navigationController: UINavigationController) { // 현재 setupNavigationBar() 로직 이동 } }src/Projects/BKDesign/Sources/Components/BKTextView.swift (1)
65-73: 에러 처리 로직의 중복을 제거할 수 있습니다.handleError() 메서드와 isError의 didSet에서 유사한 로직이 반복됩니다. 코드 중복을 줄이기 위해 리팩터링을 고려해보세요.
다음과 같이 리팩터링할 수 있습니다:
private var isError: Bool = false { didSet { - isError ? textField.setType(type: .error) : textField.setType(type: .normal) + handleError() } } private func handleError() { if isError { textField.setType(type: .error) helpMessageLabel.setColor(color: .bkContentColor(.error)) } else { textField.setType(type: .normal) helpMessageLabel.setColor(color: .bkContentColor(.tertiary)) } }src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (1)
57-60: 애니메이션 효과 추가 고려해보세요상태 변경 시 부드러운 애니메이션을 추가하면 사용자 경험이 개선됩니다.
다음과 같이 애니메이션을 추가할 수 있습니다:
@objc private func toggleCheck() { isChecked.toggle() + + // 부드러운 전환 애니메이션 + UIView.transition(with: imageView, duration: 0.15, options: .transitionCrossDissolve) { + self.updateImage() + } + sendActions(for: .valueChanged) }src/Projects/BKDesign/Sources/Components/BKSearchTextField.swift (1)
75-81: 검색 버튼 크기 제약 조건 추가를 고려해보세요버튼의 크기가 명시적으로 제한되지 않아 예상치 못한 레이아웃 문제가 발생할 수 있습니다.
다음과 같이 크기 제약 조건을 추가해보세요:
func layout() { searchButton.snp.makeConstraints { $0.trailing.equalToSuperview() .inset(BKInset.inset3) $0.centerY.equalToSuperview() + $0.width.height.equalTo(24) // 아이콘 크기에 맞는 적절한 크기 } }src/Projects/BKDesign/PreviewApp/Sources/Common/BaseViewController.swift (1)
38-44: 훅 메서드들에 문서화 추가를 권장합니다각 메서드의 목적과 호출 시점을 명확히 하면 서브클래스 개발자들에게 도움이 됩니다.
다음과 같이 문서화를 추가해보세요:
// MARK: - Common Methods +/// 뷰 계층 구조를 설정합니다. viewDidLoad에서 첫 번째로 호출됩니다. func setupView() {} +/// 뷰의 초기 속성을 구성합니다. setupView 다음에 호출됩니다. func configure() {} +/// Auto Layout 제약 조건을 설정합니다. configure 다음에 호출됩니다. func setupLayout() {} +/// UI 액션들을 바인딩합니다. setupLayout 다음에 호출됩니다. func bindAction() {} +/// 상태 변화를 바인딩합니다. bindAction 다음에 호출됩니다. func bindState() {}src/Projects/BKDesign/Sources/Components/BKBaseTextField.swift (2)
105-127: 플레이스홀더 스타일링 로직이 인상적입니다베이스라인 오프셋 계산을 통한 정밀한 텍스트 정렬이 훌륭합니다. 하지만 복잡성 때문에 코멘트 추가를 권장합니다.
다음과 같이 설명 코멘트를 추가해보세요:
func applyPlaceholderStyle() { guard let placeholderStr = placeholder, let textFont, let placeholderFont else { return } + // 텍스트와 플레이스홀더 폰트 간의 베이스라인 정렬을 위한 스타일 설정 let paragraph = NSMutableParagraphStyle() paragraph.alignment = .left paragraph.minimumLineHeight = textFont.lineHeight paragraph.maximumLineHeight = textFont.lineHeight + // 폰트 크기 차이를 보정하기 위한 베이스라인 오프셋 계산 let baseline = (textFont.capHeight - placeholderFont.capHeight) / 2
88-91: 빈 델리게이트 익스텐션 제거 고려현재 사용되지 않는 빈 익스텐션은 제거하거나 실제 필요할 때 추가하는 것이 좋습니다.
-/// 필요 시 추가 -extension BKBaseTextField: UITextFieldDelegate { - -}src/Projects/BKDesign/Sources/Components/BKLabel.swift (2)
72-81: 메서드 네이밍을 Swift 컨벤션에 맞게 개선
setText,setColor대신 Swift 스타일의 네이밍을 사용하는 것이 좋습니다.다음과 같이 변경을 고려해보세요:
-public func setText(text: String) { +public func updateText(_ text: String) { self.labelText = text apply() } -public func setColor(color: UIColor) { +public func updateColor(_ color: UIColor) { self.labelColor = color apply() }
84-101: 속성 문자열 생성 로직 단순화 고려현재 로직이 복잡하므로 단순화할 수 있습니다.
다음과 같이 단순화해보세요:
func apply() { - let baseText = fontStyle.attributedString( + let paragraphStyle = NSMutableParagraphStyle() + paragraphStyle.alignment = alignment + + attributedText = fontStyle.attributedString( from: labelText, - color: labelColor + color: labelColor, + extraAttributes: [.paragraphStyle: paragraphStyle] ) - let attributedString = NSMutableAttributedString(attributedString: baseText) - let range = NSRange(location: 0, length: attributedString.length) - let paragraphStyle = NSMutableParagraphStyle() - paragraphStyle.alignment = alignment - - attributedString.addAttribute( - .paragraphStyle, - value: paragraphStyle, - range: range - ) - - attributedText = attributedString }
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (111)
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-rectangle.imageset/checkbox-default-rectangle.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-rectangle.imageset/checkbox-default-rectangle@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-rectangle.imageset/checkbox-default-rectangle@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-round.imageset/checkbox-default-round.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-round.imageset/checkbox-default-round@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-round.imageset/checkbox-default-round@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-rectangle.imageset/checkbox-disabled-rectangle.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-rectangle.imageset/checkbox-disabled-rectangle@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-rectangle.imageset/checkbox-disabled-rectangle@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-round.imageset/checkbox-disabled-round.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-round.imageset/checkbox-disabled-round@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-round.imageset/checkbox-disabled-round@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled-rectangle.imageset/checkbox-filled-rectangle.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled-rectangle.imageset/checkbox-filled-rectangle@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled-rectangle.imageset/checkbox-filled-rectangle@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled.imageset/checkbox-filled.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled.imageset/checkbox-filled@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled.imageset/checkbox-filled@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-rectangle.imageset/checkbox-icononly_active-rectangle.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-rectangle.imageset/checkbox-icononly_active-rectangle@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-rectangle.imageset/checkbox-icononly_active-rectangle@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-round.imageset/checkbox-icononly_active-round.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-round.imageset/checkbox-icononly_active-round@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-round.imageset/checkbox-icononly_active-round@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-rectangle.imageset/checkbox-icononly_default-rectangle.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-rectangle.imageset/checkbox-icononly_default-rectangle@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-rectangle.imageset/checkbox-icononly_default-rectangle@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-round.imageset/checkbox-icononly_default-round.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-round.imageset/checkbox-icononly_default-round@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-round.imageset/checkbox-icononly_default-round@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-rectangle.imageset/checkbox-stroke-rectangle.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-rectangle.imageset/checkbox-stroke-rectangle@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-rectangle.imageset/checkbox-stroke-rectangle@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-round.imageset/checkbox-stroke-round.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-round.imageset/checkbox-stroke-round@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-round.imageset/checkbox-stroke-round@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/alert-circle.imageset/alert-circle.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/alert-circle.imageset/alert-circle@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/alert-circle.imageset/alert-circle@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/apple.imageset/apple.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/apple.imageset/apple@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/apple.imageset/apple@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bell-off.imageset/bell-off.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bell-off.imageset/bell-off@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bell-off.imageset/bell-off@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bell.imageset/bell.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bell.imageset/bell@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bell.imageset/bell@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/book-open.imageset/book-open.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/book-open.imageset/book-open@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/book-open.imageset/book-open@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bookmark.imageset/bookmark.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bookmark.imageset/bookmark@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/bookmark.imageset/bookmark@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/check.imageset/check.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/check.imageset/check@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/check.imageset/check@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-down.imageset/chevron-down.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-down.imageset/chevron-down@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-down.imageset/chevron-down@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-left.imageset/chevron-left.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-left.imageset/chevron-left@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-left.imageset/chevron-left@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-right.imageset/chevron-right.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-right.imageset/chevron-right@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-right.imageset/chevron-right@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-up.imageset/chevron-up.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-up.imageset/chevron-up@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/chevron-up.imageset/chevron-up@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit-2.imageset/edit-2.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit-2.imageset/edit-2@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit-2.imageset/edit-2@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit-3.imageset/edit-3.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit-3.imageset/edit-3@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit-3.imageset/edit-3@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit.imageset/edit.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit.imageset/edit@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/edit.imageset/edit@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/kakao.imageset/kakao.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/kakao.imageset/kakao@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/kakao.imageset/kakao@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/loader.imageset/loader.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/loader.imageset/loader@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/loader.imageset/loader@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/maximize.imageset/maximize.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/maximize.imageset/maximize@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/maximize.imageset/maximize@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/menu.imageset/menu.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/menu.imageset/menu@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/menu.imageset/menu@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/more-horizontal.imageset/more-horizontal.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/more-horizontal.imageset/more-horizontal@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/more-horizontal.imageset/more-horizontal@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/more-vertical.imageset/more-vertical.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/more-vertical.imageset/more-vertical@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/more-vertical.imageset/more-vertical@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/plus.imageset/plus.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/plus.imageset/plus@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/plus.imageset/plus@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/search.imageset/search.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/search.imageset/search@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/search.imageset/search@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/settings.imageset/settings.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/settings.imageset/settings@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/settings.imageset/settings@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/star.imageset/star.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/star.imageset/star@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/star.imageset/star@3x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/x.imageset/x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/x.imageset/x@2x.pngis excluded by!**/*.pngsrc/Projects/BKDesign/Resources/Assets.xcassets/icons/x.imageset/x@3x.pngis excluded by!**/*.png
📒 Files selected for processing (39)
.github/workflows/ios_ci.yml(1 hunks)src/Projects/BKDesign/PreviewApp/Sources/Common/BaseView.swift(1 hunks)src/Projects/BKDesign/PreviewApp/Sources/Common/BaseViewController.swift(1 hunks)src/Projects/BKDesign/PreviewApp/Sources/SceneDelegate.swift(2 hunks)src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonGroupDemoViewController.swift(1 hunks)src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonTestViewController.swift(4 hunks)src/Projects/BKDesign/PreviewApp/Sources/View/BKInputCatalogView.swift(1 hunks)src/Projects/BKDesign/PreviewApp/Sources/View/BKInputCatalogViewController.swift(1 hunks)src/Projects/BKDesign/PreviewApp/Sources/View/CatalogViewController.swift(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/AppIcon.appiconset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-rectangle.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-round.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-rectangle.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-round.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled-rectangle.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-rectangle.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-round.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-rectangle.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-round.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-rectangle.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-round.imageset/Contents.json(1 hunks)src/Projects/BKDesign/Resources/Assets.xcassets/icons/Contents.json(1 hunks)src/Projects/BKDesign/Sources/Components/BKBaseTextField.swift(1 hunks)src/Projects/BKDesign/Sources/Components/BKCheckBox.swift(1 hunks)src/Projects/BKDesign/Sources/Components/BKDivider.swift(1 hunks)src/Projects/BKDesign/Sources/Components/BKLabel.swift(1 hunks)src/Projects/BKDesign/Sources/Components/BKSearchTextField.swift(1 hunks)src/Projects/BKDesign/Sources/Components/BKTextView.swift(1 hunks)src/Projects/BKDesign/Sources/Components/Button/BKButton.swift(0 hunks)src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKIcon.swift(1 hunks)src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKInset.swift(1 hunks)src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift(2 hunks)src/Projects/BKPresentation/Sources/AuthFlow/View/LoginView.swift(1 hunks)src/Projects/Booket/Project.swift(1 hunks)src/Projects/Booket/Sources/SceneDelegate.swift(4 hunks)src/SupportingFiles/Scripts/tuist_module_init.sh(1 hunks)src/Tuist/Package.swift(0 hunks)
💤 Files with no reviewable changes (2)
- src/Projects/BKDesign/Sources/Components/Button/BKButton.swift
- src/Tuist/Package.swift
🧰 Additional context used
🧠 Learnings (31)
📓 Common learnings
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#56
File: src/Projects/BKPresentation/Sources/Constant/PresentationKeys.swift:3-5
Timestamp: 2025-07-07T13:53:13.006Z
Learning: clxxrlove prefers using enum over struct for constants namespace due to better runtime performance, memory layout, and compiler optimizations. Enum prevents instantiation at compile time without needing private init().
src/Projects/BKDesign/Resources/Assets.xcassets/icons/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/Booket/Project.swift (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-round.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-rectangle.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-round.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-rectangle.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-rectangle.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled-rectangle.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-round.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-rectangle.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-round.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Resources/Assets.xcassets/AppIcon.appiconset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonGroupDemoViewController.swift (1)
Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#56
File: src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift:109-111
Timestamp: 2025-07-07T13:47:32.784Z
Learning: The LoginViewModel's authenticateWithToken case in src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift is test code and not important for implementation according to the user.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-rectangle.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKPresentation/Sources/AuthFlow/View/LoginView.swift (2)
Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#56
File: src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift:109-111
Timestamp: 2025-07-07T13:47:32.784Z
Learning: The LoginViewModel's authenticateWithToken case in src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift is test code and not important for implementation according to the user.
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/PreviewApp/Sources/View/CatalogViewController.swift (1)
Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#56
File: src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift:109-111
Timestamp: 2025-07-07T13:47:32.784Z
Learning: The LoginViewModel's authenticateWithToken case in src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift is test code and not important for implementation according to the user.
src/Projects/BKDesign/PreviewApp/Sources/Common/BaseView.swift (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-round.imageset/Contents.json (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/Booket/Sources/SceneDelegate.swift (3)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.
src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKIcon.swift (2)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.
src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKInset.swift (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
src/Projects/BKDesign/PreviewApp/Sources/View/BKInputCatalogView.swift (2)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#68
File: src/Projects/BKDesign/Sources/Components/Button/BKButtonGroup.swift:86-92
Timestamp: 2025-07-10T08:21:16.097Z
Learning: BKButtonGroup의 horizontal 레이아웃에서 stackView.distribution을 .fillProportionally로 설정한 것은 의도된 디자인입니다. doyeonk429는 버튼들이 콘텐츠 길이에 따라 다른 너비를 가지는 것을 선호합니다.
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.
src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonTestViewController.swift (3)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#56
File: src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift:109-111
Timestamp: 2025-07-07T13:47:32.784Z
Learning: The LoginViewModel's authenticateWithToken case in src/Projects/BKPresentation/Sources/AuthFlow/ViewModel/LoginViewModel.swift is test code and not important for implementation according to the user.
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#68
File: src/Projects/BKDesign/Sources/Components/Button/BKButtonGroup.swift:86-92
Timestamp: 2025-07-10T08:21:16.097Z
Learning: BKButtonGroup의 horizontal 레이아웃에서 stackView.distribution을 .fillProportionally로 설정한 것은 의도된 디자인입니다. doyeonk429는 버튼들이 콘텐츠 길이에 따라 다른 너비를 가지는 것을 선호합니다.
src/Projects/BKDesign/Sources/Components/BKDivider.swift (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.
src/Projects/BKDesign/Sources/Components/BKBaseTextField.swift (2)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BottomSheetShadow.swift:5-10
Timestamp: 2025-07-08T17:16:15.942Z
Learning: doyeonk429는 BottomSheetShadow struct에서 apply(to:)와 asCALayerShadow() 메서드를 처음에 만들었지만 활용 측면에서 애매할 것 같아서 제외했다고 함. 정적 속성만으로 구현하는 것을 선호함.
src/Projects/BKDesign/Sources/Components/BKLabel.swift (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.
src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (1)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
🧬 Code Graph Analysis (3)
src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonGroupDemoViewController.swift (1)
src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonTestViewController.swift (1)
viewWillAppear(22-26)
src/Projects/BKDesign/PreviewApp/Sources/Common/BaseView.swift (1)
src/Projects/BKDesign/PreviewApp/Sources/Common/BaseViewController.swift (3)
setupView(39-39)setupLayout(41-41)configure(40-40)
src/Projects/BKDesign/Sources/Components/BKBaseTextField.swift (2)
src/Projects/BKDesign/Sources/Extensions/UIColor+.swift (3)
bkBackgroundColor(38-50)bkContentColor(52-63)bkBorderColor(65-76)src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift (1)
attributedString(322-342)
⏰ 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 (48)
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled.imageset/Contents.json (1)
4-17: 체크박스 채워진 이미지 리소스 확인 완료저장소 내에 checkbox-filled.png, checkbox-filled@2x.png, checkbox-filled@3x.png 모두 존재함을 셸 스크립트로 검증했습니다. Contents.json에 정의된 리소스가 누락되지 않아 런타임 크래시는 발생하지 않습니다.
src/Projects/BKDesign/Resources/Assets.xcassets/icons/Contents.json (1)
1-6: 상위 카탈로그 메타만 존재합니다 – 아이콘 세트 추가 예정인지 확인 필요
icons카탈로그에 하위*.imageset이 아직 없는 상태입니다.
향후 아이콘을 추가할 계획이 명확하다면 괜찮지만, 당장 사용처가 없다면 디렉터리를 빈 채로 두는 것은 혼란을 줄 수 있습니다. 필요 시 초기 세트를 함께 커밋하는 편이 좋습니다.src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/Contents.json (1)
1-6: 카탈로그 루트 메타데이터 추가 완료 – 문제없습니다.src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-round.imageset/Contents.json (1)
4-17: 확인 완료: checkbox-icononly_default-round 이미지 리소스가 모두 존재합니다.
- src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-round.imageset 디렉토리에
• checkbox-icononly_default-round.png
• checkbox-icononly_default-round@2x.png
• checkbox-icononly_default-round@3x.png
파일이 모두 확인되었습니다.src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_default-rectangle.imageset/Contents.json (1)
1-23: 체크박스 아이콘 에셋 카탈로그 구조가 올바르게 구성되었습니다.표준 에셋 카탈로그 형식을 따르고 있으며, 1x, 2x, 3x 스케일 변형이 모두 포함되어 있습니다. 파일명 규칙도 일관성 있게 적용되었습니다.
src/Projects/Booket/Project.swift (1)
12-12: 리소스 포함 패턴 확장이 적절합니다.글로브 패턴을
Projects/BKDesign/Resources/**로 변경하여 폰트뿐만 아니라 새로 추가된 체크박스 에셋 카탈로그를 포함한 모든 BKDesign 리소스를 포함하도록 했습니다. 새로운 디자인 시스템 컴포넌트들을 지원하기 위한 적절한 변경입니다.src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-rectangle.imageset/Contents.json (1)
1-23: 활성 상태 체크박스 아이콘 에셋 카탈로그가 올바르게 구성되었습니다.표준 에셋 카탈로그 형식을 따르고 있으며, 활성 상태를 나타내는 명확한 네이밍 규칙이 적용되었습니다. 다른 체크박스 에셋들과 일관성 있는 구조입니다.
src/Projects/BKPresentation/Sources/AuthFlow/View/LoginView.swift (1)
53-55: BKImage 시스템으로의 마이그레이션이 적절하게 수행되었습니다.이전의
BKIcon.kakaoLogo.image에서 조건부 가드를 사용하던 방식에서BKImage.Icon.kakao를 직접 사용하는 방식으로 변경되어 코드가 더 간결해졌습니다. 새로운 BKImage 시스템의 타입 안전성을 활용한 좋은 개선입니다.src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-rectangle.imageset/Contents.json (1)
1-23: 기본 상태 체크박스 에셋 카탈로그가 일관성 있게 구성되었습니다.다른 체크박스 에셋 카탈로그들과 동일한 표준 형식을 따르고 있으며, 기본 상태를 나타내는 명확한 네이밍이 적용되었습니다. 전체 체크박스 UI 시스템의 일관성을 유지하고 있습니다.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-default-round.imageset/Contents.json (1)
1-23: 표준 에셋 카탈로그 구조가 올바르게 구성되었습니다.iOS 에셋 카탈로그의 표준 구조를 따르고 있으며, 1x, 2x, 3x 해상도 변형을 모두 포함하고 있습니다. 파일명 규칙도 올바르게 적용되어 있습니다.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-disabled-rectangle.imageset/Contents.json (1)
1-23: 비활성화 체크박스 에셋이 표준에 맞게 구성되었습니다.다른 체크박스 에셋과 동일한 구조를 따르고 있어 일관성이 유지되고 있습니다.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-rectangle.imageset/Contents.json (1)
1-23: 스트로크 체크박스 에셋이 올바르게 정의되었습니다.다른 체크박스 에셋들과 일관된 구조를 유지하고 있어 전체적인 에셋 관리가 체계적입니다.
src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonGroupDemoViewController.swift (1)
19-22: 네비게이션 타이틀 설정이 올바르게 구현되었습니다.
super.viewWillAppear(animated)호출 후 네비게이션 타이틀을 설정하는 패턴은 iOS 모범 사례를 따르고 있으며, 관련 코드 스니펫에서 보여준BKButtonTestViewController와도 일관된 구조입니다.src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-filled-rectangle.imageset/Contents.json (1)
1-23: 채워진 체크박스 에셋이 표준에 맞게 구성되었습니다.모든 체크박스 에셋 파일들이 일관된 구조를 가지고 있어 유지보수성이 좋습니다. 검색된 학습 내용에 따르면 BKDesign 프로젝트에서 이미지 에셋들이 올바르게 포함되어 있다고 확인되었습니다.
src/Projects/BKDesign/PreviewApp/Sources/View/BKInputCatalogViewController.swift (1)
5-10: 깔끔한 뷰 컨트롤러 구현입니다.BaseViewController를 상속받아 타입 안전성을 보장하고, viewWillAppear에서 네비게이션 타이틀을 설정하는 구조가 적절합니다. final 키워드 사용으로 상속을 방지한 것도 좋은 선택입니다.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-icononly_active-round.imageset/Contents.json (1)
1-23: 표준 에셋 카탈로그 형식을 준수합니다.체크박스 아이콘의 모든 스케일 변형(1x, 2x, 3x)이 적절히 정의되어 있고, 파일명 규칙이 일관적입니다. 이전 학습 내용에 따르면 BKDesign 프로젝트의 이미지 에셋들이 모두 적절히 제공되고 있습니다.
src/Projects/BKDesign/Resources/Assets.xcassets/checkbox/checkbox-stroke-round.imageset/Contents.json (1)
1-23: 표준 에셋 카탈로그 형식을 준수합니다.체크박스 stroke round 아이콘의 모든 스케일 변형(1x, 2x, 3x)이 적절히 정의되어 있고, 파일명 규칙이 일관적입니다. 다른 체크박스 아이콘들과 동일한 구조를 유지하고 있어 일관성이 좋습니다.
src/Projects/BKDesign/PreviewApp/Sources/View/CatalogViewController.swift (1)
25-31: 뷰 컨트롤러 구조가 잘 설계되었습니다.viewDidLoad에서 설정과 바인딩을 분리하여 호출하는 구조가 깔끔합니다. 타이틀 설정과 배경색 설정도 적절합니다.
src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift (1)
322-342: 속성 확장 기능 추가 우수
extraAttributes매개변수 추가로 텍스트 스타일링의 유연성이 크게 향상되었습니다. 특히 다음 부분이 잘 구현되었습니다:
- 기본값 제공: 빈 딕셔너리 기본값으로 호환성 유지
- 적절한 병합 전략:
attributes.merge(extraAttributes) { $1 }로 추가 속성이 우선되도록 구현- 명확한 매개변수 순서: 기본 속성 먼저, 추가 속성 나중
이 변경으로 BKLabel, BKCheckBox 등 새로운 컴포넌트들에서 더 풍부한 텍스트 스타일링이 가능해졌습니다.
src/Projects/Booket/Sources/SceneDelegate.swift (2)
5-5: BKDesign 모듈 import 추가 적절디자인 시스템 구성요소 사용을 위한 import 추가가 적절합니다.
25-25: 네비게이션 바 설정 호출 위치 적절window 설정 전에 네비게이션 바를 구성하는 것이 올바른 타이밍입니다.
src/Projects/BKDesign/Sources/Components/BKDivider.swift (2)
5-28: Divider 컴포넌트 설계 우수BKDivider 구현이 매우 잘 설계되었습니다:
- 타입 기반 스타일링: DividerType enum으로 명확한 구분
- 적절한 intrinsic content size: 높이는 타입별, 너비는 유연하게 설정
- 디자인 시스템 일관성:
.bkDividerColor활용- 스토리보드 초기화 방지: 다른 컴포넌트들과 일관된 패턴
30-48: 타입 속성 확장 구현 깔끔DividerType의 computed properties로 높이와 색상을 관리하는 방식이 깔끔합니다. 각 타입별로 적절한 높이(medium: 8pt, small: 1pt)와 해당하는 디자인 시스템 색상을 제공합니다.
src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonTestViewController.swift (4)
24-24: 네비게이션 타이틀 설정이 적절합니다.viewWillAppear에서 네비게이션 타이틀을 설정하는 것은 적절한 위치입니다. 사용자가 뷰에 진입할 때마다 타이틀이 올바르게 표시됩니다.
62-64: BKImage.Icon 사용법이 올바르게 적용되었습니다.기존 BKIcon에서 새로운 BKImage.Icon으로의 변경이 일관성 있게 적용되었습니다. 직접 UIImage를 반환하는 방식으로 변경되어 더 직관적입니다.
79-80: 메서드 시그니처 변경이 적절합니다.UIImage? 타입으로 변경하여 타입 안전성을 높였습니다. 옵셔널 파라미터로 기본값을 제공하는 것도 좋은 설계입니다.
107-108: setupIndependentButtons 메서드의 아이콘 할당이 올바릅니다.배열의 인덱스를 통한 아이콘 할당이 올바르게 적용되었습니다. BKImage.Icon 사용법이 일관성 있게 유지되고 있습니다.
src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKIcon.swift (3)
5-8: 문서화 주석이 명확하고 유용합니다.Tuist 자동 생성 에셋 사용법과 네임스페이스 구성에 대한 설명이 명확합니다. 개발자들이 에셋 추가 시 어떻게 분류해야 하는지 알 수 있습니다.
9-36: Icon 네임스페이스 구성이 체계적입니다.BKDesignAsset에서 자동 생성된 에셋들을 체계적으로 분류하고 있습니다. 정적 프로퍼티로 UIImage를 직접 제공하여 사용이 간편합니다.
38-51: Checkbox 네임스페이스가 체계적으로 구성되었습니다.체크박스 관련 에셋들이 상태별, 타입별로 명확하게 분류되어 있습니다. 네이밍 컨벤션이 일관성 있게 적용되었습니다.
src/Projects/BKDesign/Sources/Components/BKTextView.swift (3)
14-18: 에러 상태 관리 로직이 잘 구현되었습니다.didSet을 사용한 에러 상태 변경 처리가 깔끔합니다. 상태 변경 시 텍스트 필드의 타입이 자동으로 업데이트되어 일관성이 유지됩니다.
20-25: intrinsic content size 구현이 적절합니다.고정 높이(108pt)를 제공하면서 너비는 유연하게 처리하는 것이 적절합니다. 컴포넌트의 예측 가능한 레이아웃을 보장합니다.
35-49: 초기화 패턴이 iOS 컨벤션을 잘 따릅니다.옵셔널 파라미터로 기본값을 제공하고, 프로퍼티 초기화 후 setup() 메서드를 호출하는 패턴이 적절합니다.
src/Projects/BKDesign/PreviewApp/Sources/View/BKInputCatalogView.swift (4)
7-7: BaseView 상속이 적절합니다.BaseView를 상속하여 일관된 뷰 생명주기 관리를 따르고 있습니다. 프로젝트의 구조적 일관성을 유지합니다.
9-15: 스크롤 뷰 설정이 적절합니다.세로 스크롤만 허용하고 가로 스크롤 인디케이터를 숨기는 설정이 UI 데모용으로 적절합니다. alwaysBounceVertical 설정도 좋습니다.
36-43: 체크박스 스택 뷰 구성이 합리적입니다.가로 배치된 체크박스들과 스페이서를 사용하여 적절한 레이아웃을 구성했습니다. distribution을 .equalCentering으로 설정한 것이 적절합니다.
128-131: 스크롤 뷰 제약 조건이 올바릅니다.contentLayoutGuide와 frameLayoutGuide를 적절히 사용하여 스크롤 뷰의 콘텐츠 크기와 프레임 크기를 올바르게 설정했습니다.
src/Projects/BKDesign/PreviewApp/Sources/SceneDelegate.swift (5)
8-8: 네비게이션 컨트롤러 인스턴스 프로퍼티 선언이 적절합니다.씬 전반에서 네비게이션 컨트롤러를 재사용할 수 있도록 인스턴스 프로퍼티로 선언한 것이 좋은 설계입니다.
17-23: 씬 설정이 체계적으로 구성되었습니다.네비게이션 바 설정, 윈도우 설정, 루트 뷰 컨트롤러 푸시 순서가 논리적으로 잘 구성되어 있습니다.
25-35: 네비게이션 바 기본 스타일링이 포괄적입니다.타이틀 텍스트 속성, 배경색, 그림자 제거 등 기본적인 스타일링이 잘 구성되어 있습니다. BKTextStyle과 색상 시스템을 일관성 있게 사용하고 있습니다.
41-48: 백 버튼 커스터마이징이 상세하게 구현되었습니다.백 버튼 텍스트를 숨기고 커스텀 아이콘을 사용하며, 적절한 인셋을 적용한 것이 좋습니다. withAlignmentRectInsets를 사용한 위치 조정도 정확합니다.
50-54: 네비게이션 바 appearance 적용이 완전합니다.standardAppearance, scrollEdgeAppearance, compactAppearance 모두에 동일한 스타일을 적용하여 일관성을 보장했습니다. isTranslucent를 false로 설정한 것도 적절합니다.
src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (2)
6-11: 체크박스 타입 구현이 잘 되어있습니다enum을 통한 타입 정의가 깔끔하고, 향후 확장성도 좋습니다.
15-19: 상태 관리 로직이 적절합니다
didSet옵저버를 통한 이미지 업데이트가 반응형으로 잘 구현되어 있습니다.src/Projects/BKDesign/Sources/Components/BKSearchTextField.swift (1)
6-8: 구현 방식 선택에 대한 좋은 코멘트입니다UISearchBar vs TextField 구현 방식에 대한 명확한 설명이 도움이 됩니다.
src/Projects/BKDesign/PreviewApp/Sources/Common/BaseViewController.swift (1)
5-18: 제네릭 베이스 컨트롤러 구현이 훌륭합니다타입 안정성과 재사용성을 모두 확보한 좋은 설계입니다.
src/Projects/BKDesign/Sources/Components/BKBaseTextField.swift (1)
6-11: 텍스트 필드 타입 시스템이 잘 설계되었습니다세 가지 상태(normal, brand, error)를 명확히 구분한 좋은 설계입니다.
src/Projects/BKDesign/Sources/Components/BKLabel.swift (1)
20-36: 프로퍼티 접근 제한이 훌륭한 설계입니다
@available(*, unavailable)애노테이션을 통한 API 제어가 우수합니다.
doyeonk429
reviewed
Jul 10, 2025
doyeonk429
approved these changes
Jul 10, 2025
Member
doyeonk429
left a comment
doyeonk429
left a comment
There was a problem hiding this comment.
수고하셨습니다 ! checkbox는 여유있을 때 제가 조금 수정하겠습니다 !
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (1)
63-71: 접근성 값 업데이트가 누락되었습니다
updateImage()메서드에서 이미지를 업데이트할 때 접근성 값도 함께 업데이트해야 합니다. 현재는 초기 설정에서만 접근성 값이 설정되고, 체크박스 상태가 변경될 때는 업데이트되지 않습니다.private func updateImage() { if isEnabled { imageView.image = isChecked ? type.checkedImage : type.defaultImage } else { imageView.image = type.disabledImage } + updateAccessibilityValue() }
🧹 Nitpick comments (1)
src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (1)
80-82: 메서드 가시성 일관성을 개선하세요
updateAccessibilityValue()메서드가 private extension 내에 있지만private키워드가 명시되어 있지 않습니다. 일관성을 위해private키워드를 추가하는 것이 좋습니다.- func updateAccessibilityValue() { + private func updateAccessibilityValue() { accessibilityValue = isChecked ? "선택됨" : "선택 안됨" }
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
src/Projects/BKDesign/Sources/Components/BKBaseTextField.swift(1 hunks)src/Projects/BKDesign/Sources/Components/BKCheckBox.swift(1 hunks)src/Projects/BKDesign/Sources/Components/BKLabel.swift(1 hunks)src/Projects/BKDesign/Sources/Components/BKSearchTextField.swift(1 hunks)src/Projects/BKPresentation/Sources/Common/Component/BaseViewController.swift(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- src/Projects/BKPresentation/Sources/Common/Component/BaseViewController.swift
🚧 Files skipped from review as they are similar to previous changes (3)
- src/Projects/BKDesign/Sources/Components/BKSearchTextField.swift
- src/Projects/BKDesign/Sources/Components/BKBaseTextField.swift
- src/Projects/BKDesign/Sources/Components/BKLabel.swift
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#69
File: src/Projects/BKDesign/PreviewApp/Sources/View/CatalogViewController.swift:57-70
Timestamp: 2025-07-10T14:01:04.899Z
Learning: clxxrlove는 Reed-iOS 프로젝트에서 테스트/데모용 코드와 프로덕션 코드에 대해 다른 코드 품질 기준을 적용한다. 테스트용 뷰에서는 안전성 검증 등의 엄격한 코드 품질 요구사항을 적용하지 않는 것을 선호한다.
Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#69
File: src/Projects/BKDesign/Resources/Assets.xcassets/AppIcon.appiconset/Contents.json:1-35
Timestamp: 2025-07-10T13:26:07.507Z
Learning: clxxrlove는 Reed-iOS 프로젝트에서 AppIcon 에셋 구조를 미리 설정하고, 실제 이미지 파일이 준비되기 전까지는 filename 속성을 의도적으로 생략하는 개발 방식을 사용한다. 이는 에셋 구조를 먼저 만들고 나중에 이미지를 추가하는 단계적 개발 접근법이다.
Learnt from: clxxrlove
PR: YAPP-Github/Reed-iOS#56
File: src/Projects/BKPresentation/Sources/Constant/PresentationKeys.swift:3-5
Timestamp: 2025-07-07T13:53:13.006Z
Learning: clxxrlove prefers using enum over struct for constants namespace due to better runtime performance, memory layout, and compiler optimizations. Enum prevents instantiation at compile time without needing private init().
src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (2)
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.
Learnt from: doyeonk429
PR: YAPP-Github/Reed-iOS#63
File: src/Projects/BKDesign/Sources/Extensions/UIColor+.swift:50-92
Timestamp: 2025-07-08T17:17:37.703Z
Learning: BKDesign 프로젝트에서 UIColor+ 확장의 시맨틱 컬러 메서드들(.bkContentColor, .bkBorderColor, .bkDividerColor, .bkBaseColor)이 현재 다크 모드에서도 라이트 모드 색상을 반환하는 것은 의도적인 구현입니다. 아직 다크 모드가 구현되지 않았지만, 미래의 다크 모드 지원을 위한 코드 구조로 준비되어 있습니다.
⏰ 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 (2)
src/Projects/BKDesign/Sources/Components/BKCheckBox.swift (2)
6-48: 체크박스 컴포넌트 구현이 잘 되어 있습니다4가지 스타일을 지원하는 체크박스 컴포넌트가 깔끔하게 구현되었습니다. UIControl을 상속받아 적절한 이벤트 처리를 하고 있고, 접근성 지원도 추가되었습니다.
89-120: 이미지 제공 로직이 체계적으로 구성되어 있습니다CheckboxType extension에서 각 타입별로 적절한 이미지를 제공하는 로직이 잘 구성되어 있습니다. BKImage.Checkbox를 통해 타입 안전한 이미지 참조를 사용하고 있어 좋습니다.
This was referenced Jul 27, 2025
Merged
Merged
Merged
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 관련 이슈
📘 작업 유형
📙 작업 내역
🧪 테스트 내역
🎨 스크린샷 또는 시연 영상 (선택)
Simulator.Screen.Recording.-.iPhone.16.-.2025-07-10.at.22.04.37.mp4
💬 추가 설명 or 리뷰 포인트 (선택)
Summary by CodeRabbit
신규 기능
버그 수정
리팩터
스타일
문서 및 리소스