Skip to content

fix: 온보딩 디자인 QA 반영#282

Merged
seoyoon513 merged 5 commits intodevelopfrom
BOOK-510-fix/#279
Feb 26, 2026
Merged

fix: 온보딩 디자인 QA 반영#282
seoyoon513 merged 5 commits intodevelopfrom
BOOK-510-fix/#279

Conversation

@seoyoon513
Copy link
Contributor

@seoyoon513 seoyoon513 commented Feb 20, 2026

🔗 관련 이슈

📙 작업 설명

  • 온보딩 이미지 잘리는 이슈 수정
  • 온보딩 버튼 문구 수정 (다음 -> 시작하기)
  • 온보딩 텍스트 하이라이트 적용
  • 온보딩 세 번째 이미지 리소스 교체

🧪 테스트 내역 (선택)

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

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

Screenshot_20260220_153438

Summary by CodeRabbit

  • 신규 기능

    • 온보딩 화면 세 번째 페이지에 강조 텍스트("공유") 추가
  • 스타일

    • 마지막 버튼 텍스트를 "다음"에서 "시작하기"로 변경
    • 온보딩 이미지 레이아웃 최적화 (너비 채움으로 조정)

@github-actions github-actions bot added the 🐞 fix Something isn't working label Feb 20, 2026
@github-actions github-actions bot requested a review from easyhooon February 20, 2026 06:35
@coderabbitai
Copy link

coderabbitai bot commented Feb 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0167087 and f616688.

📒 Files selected for processing (1)
  • feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/component/OnboardingPage.kt

Walkthrough

온보딩 화면의 이미지 레이아웃 제약을 제거하고, 세 번째 페이지에 강조 텍스트 리소스(highlightTextRes)를 추가했으며, 마지막 호출-투-액션 버튼 텍스트를 다음에서 시작하기로 변경했습니다.

Changes

Cohort / File(s) Summary
온보딩 화면 구성
feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/OnboardingUi.kt
세 번째 OnboardingPage 호출에 highlightTextRes 인자 추가; CTA 버튼 텍스트를 stringResource(R.string.start)로 변경 (다음 -> 시작하기).
OnboardingPage 컴포넌트 레이아웃
feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/component/OnboardingPage.kt
온보딩 이미지의 고정 높이(274.dp) 제거(이미지 modifier에서 height 제거, fillMaxWidth 유지). 불필요해진 dp import 제거.
문자열 리소스
feature/onboarding/src/main/res/values/strings.xml
next 리소스 제거/대체로 start("시작하기") 추가. onboarding_third_highlight_text("공유") 새로 추가.

Sequence Diagram(s)

(생성 기준 미충족 — 생략)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚀 안내 (업데이트된 섹션)

온보딩 화면의 디자인 QA 반영으로 이미지 레이아웃을 조정하고, 최종 버튼 텍스트를 '다음'에서 '시작하기'로 변경하며, 세 번째 페이지에 '공유' 강조 텍스트를 추가합니다.

📝 변경 사항 (업데이트된 섹션)

Cohort / File(s) 요약
온보딩 UI 레이아웃 및 버튼
feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/OnboardingUi.kt, feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/component/OnboardingPage.kt
세 번째 페이지에 highlightTextRes 파라미터 추가, 최종 버튼 텍스트를 '시작하기'로 변경, 온보딩 이미지 높이 제약 제거하여 레이아웃 단순화
문자열 리소스
feature/onboarding/src/main/res/values/strings.xml
next 리소스를 start("시작하기")로 변경, 새로운 강조 텍스트 리소스 onboarding_third_highlight_text("공유") 추가

🎯 코드 리뷰 예상 소요 시간 (업데이트된 섹션)

🎯 3 (Moderate) | ⏱️ ~20 minutes

🐰 토끼의 축하 시

온보딩이 반짝 빛나네 ✨
이미지가 온전히 보이고,
버튼은 '시작하기'로 깡총! 🚀
'공유'는 색으로 빛나고,
토끼도 기쁜 춤을 추네 🐇🎉

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

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.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 온보딩 디자인 QA 반영이라는 주요 변경사항을 명확하게 요약하고 있으며, 변경사항의 핵심을 잘 나타냅니다.
Linked Issues check ✅ Passed 이 PR은 #279 이슈의 모든 코딩 관련 요구사항(이미지 자르기 문제 해결, 버튼 텍스트 변경, '공유' 텍스트 강조)을 충족합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 #279의 명시된 목표와 관련이 있으며, 범위를 벗어난 변경사항은 없습니다.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch BOOK-510-fix/#279

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

Copy link
Contributor

@easyhooon easyhooon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code style check success만 하면 approve~

@seoyoon513 seoyoon513 merged commit 159104e into develop Feb 26, 2026
4 checks passed
@seoyoon513 seoyoon513 deleted the BOOK-510-fix/#279 branch February 26, 2026 10:05
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-510/fix] 온보딩 디자인 QA 반영

2 participants