feat: 2차 고도화 온보딩 화면 그래픽 및 문구 수정

[seoyoon513]

🔗 관련 이슈

• Close [BOOK-478/feat] 2차 고도화 온보딩 그래픽 변경 #237

📙 작업 설명

• 온보딩 그래픽 및 문구 수정

🧪 테스트 내역

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

📸 스크린샷 또는 시연 영상

온보딩 페이지 2	온보딩 페이지 3

Summary by CodeRabbit

• 개선사항
  • 온보딩 페이지의 텍스트 콘텐츠가 업데이트되었습니다.
  • 세 번째 온보딩 페이지의 강조 텍스트가 제거되었습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

온보딩 페이지의 하이라이트 텍스트를 선택적으로 설정 가능하도록 수정했습니다. 세 번째 페이지에서 하이라이트 텍스트 리소스를 제거하고, OnboardingPage 컴포넌트에서 highlightTextRes 파라미터를 nullable로 변경하여 조건부 렌더링을 구현했습니다.

Changes

코호트 / 파일(s)	변경 내용
온보딩 페이지 설정 feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/OnboardingUi.kt	세 번째 온보딩 페이지(인덱스 2)의 highlightTextRes 인수 제거
온보딩 페이지 컴포넌트 feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/component/OnboardingPage.kt	highlightTextRes를 Int? = null로 변경하여 선택적으로 만듦; titleText를 조건부로 계산하여 highlightTextRes 존재 여부에 따라 HighlightedText 또는 일반 AnnotatedString 사용; AnnotatedString 임포트 추가
온보딩 문자열 리소스 feature/onboarding/src/main/res/values/strings.xml	두 번째 및 세 번째 페이지의 제목과 설명 텍스트 업데이트; 하이라이트 텍스트를 "감정"으로 변경

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10분

• OnboardingPage의 파라미터 nullable 처리는 직관적인 변경
• 조건부 로직은 단순하고 명확한 패턴
• 문자열 리소스 업데이트는 콘텐츠 변경으로 자동화 검토 가능
• 변경 범위가 제한적이며 일관된 패턴

Poem

🐰 온보딩 길을 다듬으며,
하이라이트는 이제 선택사항,
세 번째 페이지는 더 단순해지고,
감정으로 피어나는 새로운 모습 ✨
변화의 봄이 왔네요!

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ 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	연결된 이슈 #237의 목표인 '2차 고도화 온보딩 그래픽 변경'이 충분히 달성되었습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 온보딩 UI 개선이라는 범위 내에서 이루어졌으며, 범위를 벗어난 변경은 없습니다.

✨ 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-478-feature/#237

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 194d574 and 9ddd667.

📒 Files selected for processing (3)

• feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/OnboardingUi.kt (0 hunks)
• feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/component/OnboardingPage.kt (3 hunks)
• feature/onboarding/src/main/res/values/strings.xml (1 hunks)

💤 Files with no reviewable changes (1)

• feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/OnboardingUi.kt

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: seoyoon513
Repo: YAPP-Github/Reed-Android PR: 46
File: feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/component/InfiniteLazyColumn.kt:83-95
Timestamp: 2025-07-14T00:46:03.843Z
Learning: seoyoon513과 팀은 한국어 주석을 선호하며, 한국어 주석을 영어로 번역하라는 제안을 하지 않아야 함

📚 Learning: 2025-07-09T01:14:29.836Z

Learnt from: seoyoon513
Repo: YAPP-Github/Reed-Android PR: 35
File: feature/login/src/main/kotlin/com/ninecraft/booket/feature/login/TermsAgreementScreen.kt:127-127
Timestamp: 2025-07-09T01:14:29.836Z
Learning: In the Reed-Android project's TermsAgreementScreen.kt, the OnTermDetailClick event is intentionally passed an empty string for the URL parameter because the actual URLs for terms detail pages haven't been decided yet. This is a temporary implementation that will be updated once the URLs are finalized.

Applied to files:

• feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/component/OnboardingPage.kt

🧬 Code graph analysis (1)

feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/component/OnboardingPage.kt (1)

core/common/src/main/kotlin/com/ninecraft/booket/core/common/utils/HighlightedText.kt (1)

• HighlightedText (11-30)

⏰ 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). (2)

• GitHub Check: Compose Stability Check
• GitHub Check: ci-build

🔇 Additional comments (3)

feature/onboarding/src/main/res/values/strings.xml (1)

6-11: 온보딩 문구 업데이트가 잘 적용되었습니다.

2페이지와 3페이지의 제목 및 설명 문구가 명확하게 수정되었고, 2페이지의 하이라이트 텍스트도 "감정"으로 일관되게 변경되었습니다. PR 목표와 일치하는 변경사항입니다.

feature/onboarding/src/main/kotlin/com/ninecraft/booket/feature/onboarding/component/OnboardingPage.kt (2)

29-29: nullable 파라미터 설계가 적절합니다.

highlightTextRes를 옵셔널로 변경하여 하이라이트가 필요 없는 페이지(3페이지)에서 유연하게 사용할 수 있도록 개선되었습니다. 기본값 null을 제공하여 호환성도 잘 유지하고 있습니다.

---

33-42: 조건부 렌더링 로직이 정확하게 구현되었습니다.

highlightTextRes의 null 여부에 따라 하이라이트 적용 여부를 분기 처리하는 로직이 깔끔하고 명확합니다. 두 경로 모두 AnnotatedString을 반환하여 일관성도 유지하고 있습니다. 모든 호출부에서 매개변수가 올바르게 처리되고 있으며, 페이지 2에서만 highlightTextRes 인자가 생략되어 기본값 null이 사용되고 있습니다.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[easyhooon]

LGTM!