Style/#104 온보딩 뷰 패딩 간격 수정

[soseoyo12]

🔗 연결된 이슈

• Closed: [Style] 온보딩 UI 수정 #104

📄 작업 내용

• 온보딩 뷰 패딩 간격 수정

구현 내용	IPhone 16 pro	IPhone 13 mini
GIF

네,,, 패딩 4px만 옮겼습니다 어푸 부탁드려요

그 화면에 두번째줄 텍스트 잘리는건 나연누나 탭바 변경사항 머지되고 수정해야할 거 같아요

[coderabbitai]

📝 Walkthrough

Walkthrough

온보딩 페이지의 UI 스타일 조정 작업으로, 스플래시 에셋 카탈로그의 Contents.json 파일을 삭제하고 OnboardingPage1 화면의 빨간색 구분선에 왼쪽 패딩 4를 추가했습니다.

Changes

응집 / 파일(s)	변경 사항 요약
에셋 카탈로그 Cherrish-iOS/Cherrish-iOS/Assets.xcassets/splash/Contents.json	스플래시 에셋 카탈로그의 Contents.json 메타데이터 파일 삭제 (6줄 제거)
온보딩 화면 UI 조정 Cherrish-iOS/Cherrish-iOS/Presentation/Feature/Onboarding/OnboardingPage1.swift	첫 번째 빨간색 구분선 직사각형에 왼쪽 패딩(leading padding) 4 추가

예상 코드 리뷰 노력

🎯 1 (Trivial) | ⏱️ ~3분

관련된 PRs

• Style/#74 스플래시 구현 #80: 같은 에셋 카탈로그 파일(Assets.xcassets/splash/Contents.json)을 수정하는 PR로, 이 PR에서 삭제하는 파일을 생성한 것으로 보여 관련성이 있습니다.

추천 리뷰어

• wotjs020708
• y-eonee
• sum130

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	PR이 #104 이슈의 패딩 간격 수정 요구사항을 부분적으로 충족했으나, 밑줄 정렬 수정(미완료 항목)이 빠져있습니다.	온보딩 UI 수정의 두 가지 요구사항 중 밑줄 정렬 수정도 함께 구현하거나, 별도 PR로 계획해야 합니다.
Out of Scope Changes check	⚠️ Warning	PR은 온보딩 뷰 패딩 수정에 집중하고 있으나, Contents.json 삭제는 이슈 #104의 요구사항과 직접적인 관련이 없어 범위 외 변경으로 보입니다.	Contents.json 삭제가 패딩 수정에 필요한 변경인지 명확히 하거나, 불필요한 경우 해당 파일 삭제를 제거하세요.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 변경 사항의 주요 내용(온보딩 뷰 패딩 간격 수정)을 명확하게 요약하고 있습니다.
Description check	✅ Passed	PR 설명이 패딩 간격 수정 작업과 관련되어 있으며, 연결된 이슈, 변경 내용, 시각적 검증(GIF 스크린샷)이 포함되어 있습니다.
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/#104-onboardingVIewUIFix

---

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.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

Cherrish-iOS/Cherrish-iOS/Presentation/Feature/Onboarding/OnboardingPage1.swift (1)

41-48: 하드코딩된 패딩 값이 반응형 레이아웃과 일관성이 없습니다.

파일 전체에서 .adjustedW와 .adjustedH를 사용하여 디바이스별 반응형 크기 조정을 하고 있는데, 새로 추가된 패딩 값 4는 하드코딩되어 있습니다. 다른 디바이스 크기에서 의도한 대로 정렬되지 않을 수 있습니다.

또한 두 번째 빨간색 구분선(Rectangle)에는 leading 패딩이 적용되지 않았는데, 의도된 디자인인지 확인이 필요합니다.

♻️ 일관성을 위한 수정 제안

                     Rectangle()
                         .fill(Color.red700)
                         .frame(width: 252.adjustedW, height: 1.4)
-                        .padding(.leading, 4)
+                        .padding(.leading, 4.adjustedW)
                     Rectangle()
                         .fill(Color.red700)	
                         .frame(width: 260.adjustedW, height: 1.4) 
                         .padding(.top, 3)

[sum130]

어푸푸푸푸푸푸~~~~~

[y-eonee]

[y-eonee]

ㄱ ㅣ ㄱ ㅣ 대.응

[soseoyo12]

ㅏ,,,,,