[FIX/#422] 바텀시트 sheetMaxWidth 기본값 추가

[Hyobeen-Park]

Related issue 🛠

• closed [FIX] 지도 바텀시트 - horizontal padding 제거 #422

Work Description ✏️

• 바텀시트에 존재하는 sheetMaxWidth 기본값을 Dp.Infinity로 수정

Screenshot 📸

as-is
to-be

Uncompleted Tasks 😅

N/A

To Reviewers 📢

오 이건 파라미터 뚫려있어서 생각보다 엄청 간단하게 해결했네요ㅎㅎ

Summary by CodeRabbit

새로운 기능

• 하단 시트 컴포넌트의 최대 너비를 조정할 수 있는 새로운 옵션이 추가되었습니다. 기본값은 제한 없음으로 설정되어 있습니다.

[coderabbitai]

요약

SpoonyAdvancedBottomSheet 컴포저블 함수에 sheetMaxWidth 파라미터(기본값: Dp.Infinity)를 추가하고, 이를 BottomSheetScaffold에 전달합니다. Dp 임포트를 추가하고 함수 시그니처를 업데이트하여 시트의 최대 너비를 제어할 수 있도록 합니다.

변경 사항

응집 / 파일	변경 요약
SpoonyAdvancedBottomSheet 시그니처 확장 app/src/main/java/com/spoony/spoony/core/designsystem/component/bottomsheet/SpoonyAdvancedBottomSheet.kt	sheetMaxWidth: Dp = Dp.Infinity 파라미터 추가. Dp 임포트 추가. 파라미터를 BottomSheetScaffold의 sheetMaxWidth에 전달하여 바텀시트의 최대 너비 제약 기능 활성화

예상 코드 리뷰 난이도

🎯 1 (Trivial) | ⏱️ ~2분

• 단순 파라미터 추가로, 기존 매서드에 새로운 선택적 파라미터만 추가됨
• 변경 범위가 단일 파일로 제한됨
• 기본값 설정으로 하위 호환성 유지

Pre-merge checks and finishing touches

❌ 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%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title Check	✅ Passed	PR 제목 "[FIX/#422] 바텀시트 sheetMaxWidth 기본값 추가"는 변경 사항을 명확하고 간결하게 설명하고 있습니다. 제목은 수정 대상 컴포넌트(바텀시트)와 추가된 변경 사항(sheetMaxWidth 기본값)을 구체적으로 언급하고 있으며, 관련 이슈 번호도 포함하고 있어 팀원들이 이력을 스캔할 때 주요 변경 사항을 빠르게 파악할 수 있습니다. 제목이 실제 코드 변경 사항과 일치하고 의도된 바를 명확히 전달하고 있습니다.
Linked Issues Check	✅ Passed	이슈 #422의 목표는 지도 바텀시트에서 의도하지 않은 horizontal padding을 제거하는 것입니다. 이 PR의 변경 사항인 sheetMaxWidth: Dp = Dp.Infinity 파라미터 추가는 시트의 최대 너비를 제어할 수 있도록 하여, 기본값을 무한대로 설정함으로써 시트가 전체 너비로 확장되게 하고 horizontal padding을 제거할 수 있는 메커니즘을 제공합니다. 코드 변경이 이슈에서 요구하는 padding 제거 목표를 구현하고 있습니다.
Out of Scope Changes Check	✅ Passed	PR의 모든 변경 사항이 이슈 #422의 목표 범위 내에 있습니다. Dp 임포트 추가, sheetMaxWidth: Dp = Dp.Infinity 파라미터 추가, 그리고 BottomSheetScaffold로 파라미터 전달하는 변경 사항 모두 horizontal padding 제거를 위한 너비 제어 메커니즘 구현과 직접적으로 관련되어 있습니다. 범위를 벗어난 불필요한 변경 사항이 없으며, 변경 사항이 최소하고 집중되어 있습니다.
Description Check	✅ Passed	PR 설명이 제공된 템플릿 구조를 모두 따르고 있습니다. "Related issue 🛠" 섹션에 종료된 이슈 번호가 명시되어 있고, "Work Description ✏️" 섹션에 변경 사항(sheetMaxWidth 기본값을 Dp.Infinity로 수정)이 설명되어 있으며, "Screenshot 📸" 섹션에 적-후 비교 이미지가 포함되어 있습니다. 또한 "Uncompleted Tasks 😅"와 "To Reviewers 📢" 섹션도 적절히 작성되어 있어 전체적으로 완성도 있는 설명이 제공되었습니다.

✨ 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/#422-delete-horizontal-padding

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4baea67 and fe634f1.

📒 Files selected for processing (1)

• app/src/main/java/com/spoony/spoony/core/designsystem/component/bottomsheet/SpoonyAdvancedBottomSheet.kt (3 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: PR Builder

🔇 Additional comments (3)

app/src/main/java/com/spoony/spoony/core/designsystem/component/bottomsheet/SpoonyAdvancedBottomSheet.kt (3)

7-7: LGTM!

새 파라미터 타입을 위한 필수 import입니다.

---

32-33: 파라미터 전달이 올바르게 구현되었습니다.

새로운 sheetMaxWidth 파라미터가 BottomSheetScaffold에 정확하게 전달되고 있습니다.

---

15-23: 구현이 정확하며 이슈 #422를 올바르게 해결합니다.

sheetMaxWidth: Dp = Dp.Infinity 파라미터가 BottomSheetScaffold에 정확히 전달되고 있습니다. 기본값으로 전체 너비 확장이 활성화되어 지도 바텀시트의 horizontal padding 제약이 제거됩니다. 기본값이 제공되므로 기존 호출 코드(MapScreen)는 자동으로 이 동작을 사용하며, 다른 바텀시트 컴포넌트들은 영향을 받지 않습니다.

---

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

[Roel4990]

오 해결하셨군요!! LGTM

[Roel4990]

이걸 추가하면 되는군요