Skip to content

Conversation

@y-eonee
Copy link
Contributor

@y-eonee y-eonee commented Jan 22, 2026

🔗 연결된 이슈

📄 작업 내용

  • 가로모드 끔
  • 스플래시 애니메이션 적용해보려고 노력해봄

@y-eonee y-eonee linked an issue Jan 22, 2026 that may be closed by this pull request
1 task
@y-eonee y-eonee requested review from a team, soseoyo12, sum130 and wotjs020708 and removed request for a team January 22, 2026 10:17
@coderabbitai
Copy link

coderabbitai bot commented Jan 22, 2026

📝 Walkthrough

Walkthrough

이 변경사항은 Xcode 프로젝트의 앱 방향 설정을 단순화하고, 여러 SwiftUI 뷰에 일관된 상단 여백(20.adjustedH)을 추가합니다. 기능적 로직 변경 없이 UI 레이아웃 조정만 포함합니다.

Changes

코호트 / 파일 변경 요약
Build Configuration
Cherrish-iOS/Cherrish-iOS.xcodeproj/project.pbxproj
디바이스별 방향 설정(iPhone, iPad) 제거 후 통합된 INFOPLIST_KEY_UISupportedInterfaceOrientations 추가. iPad 방향 문자열을 LandscapeLeft, LandscapeRight, Portrait, PortraitUpsideDown으로 업데이트 (Debug/Release 설정 반영)
Calendar Views
...Calendar/SelectTreatment/SelectTreatmentView.swift, ...Calendar/Treatment/View/NoTreatment/NoTreatmentView.swift, ...Calendar/Treatment/View/Treatment/TreatmentView.swift
네비게이션 바 상단에 Spacer(minLength: 20.adjustedH) 추가하여 일관된 상단 여백 제공
Challenge View
...ChallengeView/View/CreateChallengeView.swift
수직 스택 상단에 Spacer(minLength: 20.adjustedH) 추가하여 상단 패딩 추가

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 분

Possibly related PRs

  • Style/#102 뷰 연결 완료 #112: 동일한 SwiftUI 뷰 파일들(SelectTreatmentView.swift, NoTreatmentView.swift, TreatmentView.swift)에서 상단 여백/레이아웃 조정을 수행합니다.
  • Fix/#151 QA 반영 #155: SelectTreatmentView, NoTreatmentView, TreatmentView 등 동일한 UI 뷰 파일들의 상단 여백/레이아웃 조정을 포함합니다.
  • Feat/#52 기기대응 익스텐션 #53: adjustedH 확장 기능이 이 PR의 20.adjustedH 사용에 의존하므로 관련이 있습니다.

Suggested labels

나연🐹, Fix

Suggested reviewers

  • wotjs020708
  • soseoyo12
  • sum130
🚥 Pre-merge checks | ✅ 3 | ❌ 2
❌ Failed checks (2 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive PR 제목이 '디테일잡자'로 구체적이지 않으며, 실제 변경사항(가로모드 비활성화, UI 스페이싱 조정)을 명확히 반영하지 못합니다. PR 제목을 '가로모드 비활성화 및 네비게이션바 상단 스페이싱 추가'와 같이 구체적으로 변경하여 주요 변경사항을 명확히 표현해주세요.
Linked Issues check ❓ Inconclusive 이슈 #157은 구체적인 요구사항이 명시되지 않았으나('세부적으로 적어주세요'), PR의 변경사항(가로모드 비활성화, 네비바 스페이싱 추가)이 이슈와 연결되어 있습니다. 이슈 #157에 구체적인 요구사항 상세 내용을 추가하여 PR의 변경사항이 모든 요구사항을 충족하는지 검증할 수 있도록 해주세요.
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed PR 설명이 변경사항의 주요 내용(가로모드 비활성화, 스페이싱 추가)과 관련이 있으며, 해결한 이슈(#157)를 명시하고 있습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 scope 내에 있습니다: 프로젝트 설정(가로모드 비활성화)과 UI 레이아웃 조정(스페이싱 추가)이 이슈 #157과 관련이 있습니다.
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 fix/#157-디테일잡자

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

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

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/Calendar/SelectTreatment/SelectTreatmentView.swift (1)

15-29: body 내 Spacer가 VStack으로 감싸지지 않아 다른 뷰들과 레이아웃 동작이 다를 수 있습니다.

다른 뷰들(NoTreatmentView, TreatmentView, CreateChallengeView)은 VStack(spacing: 0)으로 전체를 감싸고 그 안에 Spacer를 배치했지만, 이 뷰는 Spacer가 body에 직접 배치되어 있습니다. 이로 인해 부모 컨테이너에 따라 레이아웃 동작이 달라질 수 있습니다.

🔧 일관성을 위한 수정 제안
     var body: some View {
+        VStack(spacing: 0) {
             Spacer()
                 .frame(height: 20.adjustedH)
             
             CherrishNavigationBar(
                 title: "시술 여부 선택",
                 leftButtonAction: {
                     calendarCoordinator.pop()
                     tabBarCoordinator.isTabbarHidden = false
                 },
                 rightButtonAction: {
                     tabBarCoordinator.isTabbarHidden = false
                     calendarCoordinator.popToRoot()
                 }
             )
             
             VStack(spacing: 0) {
                 // ... existing content
             }
+        }
     }

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.

어푸푸

Copy link
Contributor

@soseoyo12 soseoyo12 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.

좋습니당 굿굿뜨!!!

@y-eonee y-eonee merged commit 119232c into develop Jan 22, 2026
1 check passed
@y-eonee y-eonee deleted the fix/#157-디테일잡자 branch January 22, 2026 11:15
@y-eonee y-eonee self-assigned this Jan 23, 2026
@y-eonee y-eonee added 나연🐹 Fix 버그 수정 labels Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix 버그 수정 나연🐹

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 디테일잡자

5 participants