Skip to content

Comments

Style/#136 이미지 크기, 텍스트 잘림 수정#138

Merged
soseoyo12 merged 2 commits intodevelopfrom
style/#136-onboardingTextFix
Jan 21, 2026
Merged

Style/#136 이미지 크기, 텍스트 잘림 수정#138
soseoyo12 merged 2 commits intodevelopfrom
style/#136-onboardingTextFix

Conversation

@soseoyo12
Copy link
Contributor

🔗 연결된 이슈

📄 작업 내용

  • 이미지 크기 수정
  • 텍스트 잘림 해결

@soseoyo12 soseoyo12 requested a review from a team January 21, 2026 12:54
@soseoyo12 soseoyo12 self-assigned this Jan 21, 2026
@soseoyo12 soseoyo12 requested review from sum130, wotjs020708 and y-eonee and removed request for a team January 21, 2026 12:54
@soseoyo12 soseoyo12 linked an issue Jan 21, 2026 that may be closed by this pull request
1 task
Copy link
Contributor

@y-eonee y-eonee left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@sum130 sum130 left a comment

Choose a reason for hiding this comment

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

어푸푸~

TabView(selection: $currentPage) {
OnboardingPage1()
.tag(0)
.padding(.bottom, 20)
Copy link
Contributor

Choose a reason for hiding this comment

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

기기대응 플리즈용

@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 2026

📝 Walkthrough

Walkthrough

온보딩 화면의 레이아웃을 미세 조정했습니다: TabView 내 첫 페이지에 하단 패딩을 추가하고, 첫 온보딩 페이지의 오버레이 이미지 너비를 증가시켰습니다.

Changes

Cohort / File(s) 변경 요약
온보딩 컨테이너 레이아웃
Cherrish-iOS/Cherrish-iOS/Presentation/Feature/Onboarding/OnboardingContainerView.swift
TabView 내 OnboardingPage1에 하단 패딩 .padding(.bottom, 20.adjustedH) 추가
온보딩 페이지 1 이미지
Cherrish-iOS/Cherrish-iOS/Presentation/Feature/Onboarding/OnboardingPage1.swift
오버레이 이미지 Image(.illustrationOnboardingCh) 너비를 120.adjustedW에서 152.adjustedW로 변경

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • sum130
  • y-eonee
  • wotjs020708

작업 개요 및 기존 섹션 (업데이트/유지)

작업 개요

온보딩 페이지의 레이아웃을 미세 조정했습니다. OnboardingContainerView에서 첫 번째 페이지에 하단 패딩을 추가하고, OnboardingPage1의 오버레이 이미지 너비를 증가시켰습니다.

변경 사항

뭉치 / 파일 변경 요약
온보딩 레이아웃 조정
Cherrish-iOS/Cherrish-iOS/Presentation/Feature/Onboarding/OnboardingContainerView.swift
TabView 내 OnboardingPage1에 하단 패딩 20 추가
온보딩 페이지 1 이미지 크기
Cherrish-iOS/Cherrish-iOS/Presentation/Feature/Onboarding/OnboardingPage1.swift
오버레이 이미지(illustrationOnboardingCh) 너비를 120.adjustedW에서 152.adjustedW로 증가

예상 코드 리뷰 소요 시간

🎯 2 (Simple) | ⏱️ ~10 minutes

관련 PR

추천 리뷰어

  • sum130
  • y-eonee
  • wotjs020708
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 이미지 크기 및 텍스트 잘림 수정이라는 변경사항의 주요 내용을 명확하게 요약하고 있습니다.
Description check ✅ Passed PR 설명은 이미지 크기 수정과 텍스트 잘림 해결이라는 변경사항과 관련이 있으며, 연결된 이슈 #136을 명시하고 있습니다.
Linked Issues check ✅ Passed PR은 이슈 #136의 온보딩 텍스트 잘림 문제 해결을 위해 이미지 크기 조정(152.adjustedW)과 하단 패딩 추가를 구현하여 요구사항을 충족합니다.
Out of Scope Changes check ✅ Passed 모든 변경사항(이미지 너비 증가, 하단 패딩 추가)은 이슈 #136의 온보딩 화면 레이아웃 및 텍스트 렌더링 개선에 직접 관련되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 style/#136-onboardingTextFix

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

@wotjs020708 wotjs020708 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.!

TabView(selection: $currentPage) {
OnboardingPage1()
.tag(0)
.padding(.bottom, 20)
Copy link
Contributor

Choose a reason for hiding this comment

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

기기대응

@soseoyo12 soseoyo12 merged commit 28c464c into develop Jan 21, 2026
1 check passed
@soseoyo12 soseoyo12 deleted the style/#136-onboardingTextFix branch January 21, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Style] 온보딩 텍스트 잘림 문제 해결

4 participants