[FIX/#150] 검색화면 imePadding 삭제

[nhyeonii]

Related issue 🛠

• closed [FIX] 검색화면 imePadding 삭제 #150

Work Description ✏️

• 검색화면의 불필요하게 올라오는 imePadding을 제거했어요.

Uncompleted Tasks 😅

• N/A

To Reviewers 📢

Summary by CodeRabbit

• 리팩토링
  • 화면 레이아웃 구조를 개선하였습니다.
  • UI 요소의 배치와 렌더링 순서를 최적화하였습니다.
  • 진행률 표시 및 콘텐츠 표시 방식을 재정렬하였습니다.
  • 모든 기존 기능과 상호작용은 유지되었습니다.

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

[coderabbitai]

Walkthrough

ProcedureScreen에서 Scaffold 레이아웃을 최상위 Column 레이아웃으로 교체하고, TopAppBar를 상단에 추가하며, 하단 버튼을 Column의 trailing CherrishButton으로 이동. 내용 영역을 weight 기반 구조로 재구성하고 StepProgressBar를 조건부 렌더링으로 추가하며, 중첩된 조건문을 streamlined when 표현식으로 변경.

Changes

Cohort / File(s)	변경 요약
ProcedureScreen 레이아웃 재구성 app/src/main/java/com/cherrish/android/presentation/calendar/procedure/ProcedureScreen.kt	Scaffold를 Column으로 교체하고 TopAppBar (BackAndCloseTopAppBar) 추가. innerPadding 기반 레이아웃을 weight 기반 구조로 변경. StepProgressBar를 조건부 렌더링으로 추가. 중첩된 조건문을 when 표현식으로 정리. ExistenceContent, CategoryContent, RecoveryScheduleContent, FilteringContent, FilteringWithSearchContent, DowntimeContent의 배치 및 padding 의미론 변경. 하단 시트 및 downtime 상호작용 유지.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• [FIX/#144] 2차 QA 수정 사항 반영 #149: ProcedureScreen의 최상위 레이아웃과 bottom-bar/Scaffold 처리를 동일하게 수정하므로 직접 관련됨
• [FIX/#120] 시술 플로우 이슈 해결 #122: ProcedureScreen.kt와 관련 composable을 수정하며, FilteringContent/FilteringWithSearchContent의 bottomPadding 파라미터를 bottomContentPadding으로 변경하고 하단 padding 값을 조정하므로 겹침

Suggested reviewers

• sohee6989
• usuuhyn

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	⚠️ Warning	raw_summary에 따르면 변경사항이 Scaffold 레이아웃 재구성, TopAppBar 추가, Column 구조로의 변환 등 포괄적인 UI 리팩토링을 포함하고 있으며, 이는 단순한 imePadding 제거 범위를 벗어납니다.	제시된 변경사항이 imePadding 제거 목표 범위를 크게 초과하는지 명확히 하거나, 이러한 추가 변경사항이 필요한 이유를 설명해주세요.
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.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 linked issue #150의 목표인 검색 화면의 imePadding 삭제와 정확히 일치하며, 변경사항을 명확하게 설명합니다.
Description check	✅ Passed	PR 설명이 리포지토리의 템플릿을 따르고 있으며, 관련 이슈, 작업 내용, 미완료 작업 섹션이 모두 작성되어 있습니다.
Linked Issues check	✅ Passed	PR의 코드 변경사항이 linked issue #150의 목표인 검색 화면에서 불필요한 imePadding 제거와 일치합니다.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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