Skip to content

fix: 온보딩 페이지 수정사항 적용#274

Merged
clxxrlove merged 2 commits intodevelopfrom
BOOK-509-feature/#273
Feb 15, 2026
Merged

fix: 온보딩 페이지 수정사항 적용#274
clxxrlove merged 2 commits intodevelopfrom
BOOK-509-feature/#273

Conversation

@doyeonk429
Copy link
Member

@doyeonk429 doyeonk429 commented Feb 14, 2026

🔗 관련 이슈

📘 작업 유형

  • ✨ Feature (기능 추가)
  • 🐞 Bugfix (버그 수정)
  • 🔧 Refactor (코드 리팩토링)
  • ⚙️ Chore (환경 설정)
  • 📝 Docs (문서 작성 및 수정)
  • ✅ Test (기능 테스트)
  • 🎨 style (코드 스타일 수정)

📙 작업 내역

  • 이미지 에셋 교체 및 문구 수정
  • 피그마 디자인 가이드 대로 이미지 사이즈 비율 적용 및 간격 수정

🧪 테스트 내역

  • 브라우저/기기에서 동작 확인
  • 엣지 케이스 테스트 완료
  • 기존 기능 영향 없음

🎨 스크린샷 또는 시연 영상 (선택)

기능 미리보기 미리보기 미리보기
온보딩

✅ PR 체크리스트

  • 커밋 메시지가 명확합니다
  • PR 제목이 컨벤션에 맞습니다
  • 관련 이슈 번호를 작성했습니다
  • 기능이 정상적으로 작동합니다
  • 불필요한 코드를 제거했습니다

Summary by CodeRabbit

릴리스 노트

  • UI 및 콘텐츠 개선
    • 온보딩 이미지의 높이와 레이아웃이 조정되어 더 나은 시각적 구성을 제공합니다.
    • 온보딩 페이지의 제목, 설명, 강조 텍스트 등 모든 사용자 대면 콘텐츠가 새로 업데이트되었습니다.
    • 인터페이스의 여백과 간격 설정이 최적화되었습니다.

@doyeonk429 doyeonk429 self-assigned this Feb 14, 2026
@doyeonk429 doyeonk429 added the 🐞 fix Something isn't working label Feb 14, 2026
@github-actions github-actions bot requested a review from clxxrlove February 14, 2026 16:25
@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Walkthrough

온보딩 화면을 v1.4.0 버전으로 업데이트합니다. 이미지 에셋 파일명을 "graphic"에서 "onboarding2"로 변경하고, 온보딩 뷰의 텍스트 문구를 수정하며, 이미지 높이 제약 조건(너비 대비 274/375 비율)과 간격 상수를 조정합니다.

Changes

Cohort / File(s) Summary
이미지 에셋
src/Projects/BKDesign/Resources/Assets.xcassets/graphics/onboarding2.imageset/Contents.json
모든 해상도(1x, 2x, 3x)에 대해 파일명을 "graphic"에서 "onboarding2"로 변경합니다.
온보딩 뷰 UI 및 레이아웃
src/Projects/BKPresentation/Sources/OnboardingFlow/View/OnboardingView.swift
이미지 뷰 높이 제약 조건 추가(너비의 274/375), 레이블 스택 상단 간격 조정(spacing7 → spacing8), 온보딩 페이지의 제목, 설명, 강조 단어 등 텍스트 문구 업데이트입니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ⚠️ Unable to check for merge conflicts: Invalid branch name format
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 온보딩 페이지 수정사항을 명확하게 설명하며, 변경사항의 주요 목적과 일치합니다.
Linked Issues check ✅ Passed PR의 모든 변경사항이 이슈 #273의 요구사항을 충족합니다: 이미지 에셋 적용 및 텍스트 문구 수정이 완료되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 온보딩 페이지 업데이트와 관련되어 있으며, 이슈 #273의 범위 내에서 진행되었습니다.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch BOOK-509-feature/#273
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch BOOK-509-feature/#273
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
src/Projects/BKPresentation/Sources/OnboardingFlow/View/OnboardingView.swift (1)

114-118: 매직 넘버를 상수로 추출하는 것을 고려해 보세요.

274.0 / 375.0 비율이 피그마 디자인 가이드에서 온 값이라면, LayoutConstants에 명명된 상수로 추출하면 의도가 더 명확해지고 나중에 변경 시 관리가 편해집니다.

♻️ 제안
 // LayoutConstants enum 내부에 추가
 enum LayoutConstants {
+    static let imageAspectRatio: CGFloat = 274.0 / 375.0
     static let labelStackSpacing = BKSpacing.spacing3
     static let labelStackTopOffset = BKSpacing.spacing8
     static let labelStackHorizontalInset = BKInset.inset5
 }
 imageView.snp.makeConstraints {
     $0.top.equalToSuperview()
     $0.horizontalEdges.equalToSuperview()
-    $0.height.equalTo(imageView.snp.width).multipliedBy(274.0 / 375.0)
+    $0.height.equalTo(imageView.snp.width).multipliedBy(LayoutConstants.imageAspectRatio)
 }

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@clxxrlove clxxrlove merged commit 70fddb6 into develop Feb 15, 2026
5 checks passed
@clxxrlove clxxrlove deleted the BOOK-509-feature/#273 branch February 15, 2026 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BOOK-509/feat] v1.4.0 온보딩 화면 수정사항 적용

2 participants

Comments