[FIX/#134] 1차 QA 수정 사항 반영

[nhyeonii]

Related issue 🛠

• closed [FIX] 1차 QA 수정 사항 반영 #134

Work Description ✏️

• 1차 qa의 수정 사항을 반영했어요.

Screenshot 📸

Uncompleted Tasks 😅

• N/A

To Reviewers 📢

반영햇어요

Summary by CodeRabbit

릴리스 노트

• 버그 수정

  • 절차 목록 정렬 순서 개선
  • 다운타임 기간이 없는 경우 표시 로직 개선
  • 선택된 절차 시트 스크롤 동작 최적화

• UI 개선

  • 캘린더 및 검색 필드 패딩 조정
  • 절차 아이템 클릭 상호작용 개선
  • 콘텐츠 정렬 최적화

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

[coderabbitai]

📋 개요

캘린더 화면의 UI 패딩 조정, 절차 목록 순서 변경, 다운타임 모드 매개변수 제거, 그리고 다운타임 처리 로직 개선 등 QA 수정 사항들을 반영하는 변경사항입니다.

🎯 변경 내용

응집 영역 / 파일(들)	변경 요약
UI 패딩 및 정렬 조정 app/src/main/java/com/cherrish/android/presentation/calendar/CalendarScreen.kt, app/src/main/java/com/cherrish/android/presentation/calendar/component/CherrishCalendar.kt, app/src/main/java/com/cherrish/android/presentation/calendar/component/ProcedureSearchTextField.kt	CalendarScreen의 상단 패딩을 40.dp로 변경하고 하단 패딩 제거; CherrishCalendar에 수평 패딩(11.dp) 추가; ProcedureSearchTextField에 중앙 정렬 추가
절차 목록 순서 변경 app/src/main/java/com/cherrish/android/presentation/calendar/CalendarViewModel.kt, app/src/main/java/com/cherrish/android/presentation/calendar/procedure/ProcedureViewModel.kt, app/src/main/java/com/cherrish/android/presentation/calendar/procedure/component/SelectedProcedureBottomSheet.kt	CalendarViewModel에서 3개 위치의 절차 목록을 역순으로 변경; ProcedureViewModel에서 항목 추가 시 순서 변경(append → prepend); SelectedProcedureBottomSheet에서 스크롤 대상을 마지막 항목으로 변경
다운타임 모드 매개변수 제거 및 클릭 가능 로직 단순화 app/src/main/java/com/cherrish/android/presentation/calendar/component/ProcedureInfoItem.kt, app/src/main/java/com/cherrish/android/presentation/calendar/component/ProcedureScheduleCard.kt	ProcedureInfoItem에서 isDowntimeMode 매개변수 제거; 클릭 가능 상태를 항상 활성화로 변경; ProcedureScheduleCard에서 해당 매개변수 전달 제거
다운타임 처리 로직 개선 app/src/main/java/com/cherrish/android/presentation/calendar/util/ProcedureUtil.kt, app/src/main/java/com/cherrish/android/presentation/calendar/procedure/component/SelectedProcedureItem.kt	ProcedureUtil의 getProcedureType에서 다운타임 기간이 0일 때 조건부 반환 추가; SelectedProcedureItem의 다운타임 텍스트를 0일 특수 케이스 처리로 변경

📊 예상 코드 리뷰 난이도

🎯 3 (중간) | ⏱️ ~20분

🔗 관련 PR

• [FEAT/#16] 캘린더 뷰 컴포넌트 구현 #22: CalendarViewModel, ProcedureInfoItem, ProcedureScheduleCard, ProcedureUtil 등 동일 컴포넌트의 시그니처 및 로직 변경
• [FIX#81] 투두 리스트 컴포넌트 터치 영역 수정 #82: 클릭 처리 동작 제거 및 하위 컴포넌트의 클릭 관련 매개변수 제거로 유사한 패턴
• [FEAT/#45] 캘린더 API 연결 #63: CalendarViewModel.kt, ProcedureInfoItem, ProcedureUtil 등 동일 파일의 순서 지정 및 다운타임 처리 수정

👥 검토 권장 대상

• sohee6989
• usuuhyn
• hyeminililo

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

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.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목은 QA 수정 사항을 반영하는 것으로 변경 사항의 주요 목적을 명확하게 전달하고 있습니다.
Description check	✅ Passed	PR 설명이 관련 이슈, 작업 내용, 스크린샷, 미완료 작업, 리뷰어 메모를 포함하여 템플릿을 대부분 충족합니다.
Linked Issues check	✅ Passed	코드 변경 사항들(UI 패딩 조정, 리스트 순서 변경, 다운타임 모드 로직 개선)이 QA 수정 사항 반영이라는 이슈 #134의 목표를 충족합니다.
Out of Scope Changes check	✅ Passed	모든 변경 사항이 QA 수정 사항 반영이라는 이슈 범위 내에 있으며, 불필요한 외부 변경은 없습니다.

_{✏️ 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.}

[sohee6989]

수고하셨습니다!!

[usuuhyn]

수고하셧습니다 !!!