[FIX/#144] 2차 QA 수정 사항 반영

[nhyeonii]

Related issue 🛠

• closed [FIX] 2차 QA 수정 #144

Work Description ✏️

• 2차 QA 수정 사항을 반영했어요

Screenshot 📸

Uncompleted Tasks 😅

• N/A

To Reviewers 📢

Summary by CodeRabbit

• 새로운 기능

  • 시술 진행 화면과 온보딩 화면에 하단 작업 버튼(완료/다음) 추가
  • 회복 일정 입력 시 포커스 기반 자동 스크롤(입력 시 화면으로 자동 이동) 추가

• UI/UX 개선

  • 달력·시술 예약 레이아웃 간격을 비율 기반으로 재구성해 반응성 향상
  • 선택 항목 그리드 정렬 개선(2열 균등 분배)
  • 다운타임 표시 문구 조건부 표시 개선

• 개선사항

  • 입력 포커스 및 키보드 처리와 스크롤/패딩 조정 개선

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

[coderabbitai]

Warning

Rate limit exceeded

@nhyeonii has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 20 minutes and 35 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Walkthrough

여러 화면의 레이아웃을 리팩토링: Scaffold 도입, 하단 액션 바 추가, 고정 스페이싱을 weight 기반 스페이서로 전환, 포커스 시 자동 스크롤(BringIntoViewRequester) 추가, LazyVerticalGrid를 수동 Row 기반 2열 그리드로 대체함.

Changes

Cohort / File(s)	변경 요약
달력 화면 및 카드 레이아웃 app/src/main/java/com/cherrish/android/presentation/calendar/CalendarScreen.kt, app/src/main/java/com/cherrish/android/presentation/calendar/component/ProcedureScheduleCard.kt	외부에 하단 padding(10.dp) 추가, 카드/리스트에 weight(1f) 적용 및 고정 top/bottom 패딩을 weight 기반 Spacer로 전환하여 수직 분배 방식으로 변경
절차 화면 리팩토링 (메인 흐름 변경) app/src/main/java/com/cherrish/android/presentation/calendar/procedure/ProcedureScreen.kt, app/src/main/java/com/cherrish/android/presentation/calendar/procedure/component/SelectedProcedureBottomSheet.kt, .../DowntimeBottomSheet.kt	최상위 Column → Scaffold 전환, 하단 바(단계별 CherrishButton) 추가, StepProgressBar 조건부 렌더링 유지, RecoverySchedule 등에 verticalScroll/innerPadding/imePadding 적용해 내부 레이아웃 재구성
회복 스케줄: 포커스/BringIntoView 추가 app/src/main/java/com/cherrish/android/presentation/calendar/procedure/content/RecoveryScheduleContent.kt	BringIntoViewRequester 및 코루틴 기반 bringIntoView 로직 추가, ScheduleSettingSection/DateInputBasicSection에 onFocus: () -> Unit 파라미터 전달 및 포커스시 자동 스크롤 트리거
선택 섹션 그리드 구현 변경 app/src/main/java/com/cherrish/android/presentation/calendar/procedure/component/SelectionSection.kt	LazyVerticalGrid 제거하고 Row 기반 수동 2열 그리드(함수명 ChipGrid → ChipFlowGrid)로 변경; 각 칩에 Modifier.weight(1f) 적용, 짝수/홀수 항목 정렬 보정
온보딩 화면 레이아웃/포커스 처리 변경 app/src/main/java/com/cherrish/android/presentation/onboarding/information/OnboardingInformationScreen.kt	Column → Scaffold 전환, 하단 Next 버튼 바에 배치, onDoneAction을 코루틴 지연으로 비동기 처리해 키보드 숨김 후 포커스 해제
다운타임 텍스트 조건 처리 app/src/main/java/com/cherrish/android/presentation/calendar/procedure/component/DowntimeBottomSheet.kt	min/max 다운타임이 둘다 0일 때 표시 텍스트를 "보통 다운타임 0일"로 조건부 변경

Sequence Diagram(s)

(생성 조건 미충족 — 생략)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Possibly related PRs

• [FEAT/#67] 시술 뷰 UI 구현 #84 — ProcedureScreen, RecoveryScheduleContent, ProcedureScheduleCard 등 동일 컴포넌트 집합을 광범위하게 수정한 PR으로 코드 수준 연결이 강함.
• [FIX#145] 2차 QA 수정 사항 반영 #146 — RecoveryScheduleContent의 포커스/bring-into-view 관련 변경을 다룬 PR로, 포커스 이동 로직과 밀접하게 관련됨.
• [FEAT/#76] 정보입력뷰 api 연결 #85 — OnboardingInformationScreen의 IME/포커스 처리 변경을 포함한 PR로 온보딩 화면 관련 변경과 연관됨.

Suggested reviewers

• sohee6989
• usuuhyn
• hyeminililo

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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.
Title check	❓ Inconclusive	제목이 관련 이슈 #144의 주요 내용(2차 QA 수정)을 반영하고 있으나, 구체적인 변경 사항의 세부 내용을 명확히 나타내지 못하고 있습니다.	제목을 더 구체적으로 수정하여 실제 변경 내용(레이아웃 개선, 스크롤 추가 등)을 반영하거나, 현재 제목으로도 허용 가능한 범위 내에서 일관성을 유지해야 합니다.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description check	✅ Passed	PR 설명이 템플릿의 주요 섹션을 포함하고 있으나, Work Description이 매우 간단하고 구체적인 작업 내용 설명이 부족합니다.
Linked Issues check	✅ Passed	코드 변경사항이 이슈 #144의 '2차 QA 수정'이라는 목표와 일치하며, 6개 파일의 UI/레이아웃 개선이 반영되어 있습니다.
Out of Scope Changes check	✅ Passed	모든 코드 변경사항이 캘린더 화면의 UI/레이아웃 개선, 스크롤 처리, 버튼 배치 등 QA 수정과 관련된 범위 내에 있습니다.

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

---

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

[hyeminililo]

넘넘따리 수고했뜹니다

[sohee6989]

수고하셨습니다!!