업로드 관련 QA 수정

[DongChyeon]

🛠 Related issue

없음

어떤 변경사항이 있었나요?

• 🐞 BugFix Something isn't working
• 🎨 Design Markup & styling
• 📃 Docs Documentation writing and editing (README.md, etc.)
• ✨ Feature Feature
• 🔨 Refactor Code refactoring
• ⚙️ Setting Development environment setup
• ✅ Test Test related (Junit, etc.)

✅ CheckPoint

PR이 다음 요구 사항을 충족하는지 확인하세요.

• PR 컨벤션에 맞게 작성했습니다. (필수)
• merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
• Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
• BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

✏️ Work Description

• currentState 가 Hidden이 아닌 동안은 보여주도록BottomSheet 커스텀 dim 색상 처리
• 업로드 후 내 투표 탭으로 이동

😅 Uncompleted Tasks

N/A

📢 To Reviewers

📃 RCA 룰

• R: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
• C: 웬만하면 반영해 주세요. (Comment)
• A: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

Summary by CodeRabbit

릴리즈 노트

• 개선 사항

  • 하단 시트의 배경 어두워짐 효과가 더욱 부드럽고 자연스러운 애니메이션으로 개선되었습니다.

• 버그 수정

  • 홈 화면에서 초기 탭 선택이 올바르게 작동하도록 수정되었습니다.

[coderabbitai]

Walkthrough

하단 시트의 Material3 기본 scrim을 animateFloatAsState로 구동되는 커스텀 애니메이션 전체 화면 어두움 오버레이로 교체했고, Canvas로 사각형을 그려 구현했습니다. 또한 홈 화면의 초기 탭 선택 로직을 하드코딩된 문자열 대신 HomeTab.MY_FEED.name과 비교하도록 수정했습니다.

Changes

Cohort / File(s)	Summary
Bottom Sheet 커스텀 오버레이 core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/BottomSheet.kt	Material3 기본 scrim을 animateFloatAsState 기반의 커스텀 Canvas 오버레이로 교체, dimAlpha 상태 애니메이션 추가, Box(fillMaxSize()) 레이아웃 래핑 및 네비게이션 바 스페이서 위치 재조정
홈 네비게이션 탭 선택 로직 feature/home/src/main/java/com/sseotdabwa/buyornot/feature/home/navigation/HomeNavigation.kt	initialTab 선택 로직을 하드코딩된 "REVIEW" 문자열 비교에서 HomeTab.MY_FEED.name 기반 비교로 변경

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Feat/#76 디자인 QA 수정 #78: 하단 시트의 scrim/어두움 동작 및 레이아웃 변경 사항이 모두 포함되어 있어 관련성이 높음
• Feat/#15 피드 업로드 화면 UI 구현 #21: 동일한 BottomSheet.kt 컴포넌트의 scrim/오버레이 렌더링 로직 변경으로 관련성이 높음

Suggested labels

✨ FEAT, 💪 동현동현동현

Suggested reviewers

• Imagine-Choi

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목 '업로드 관련 QA 수정'은 실제 변경사항의 주요 부분과 관련이 있습니다. PR에서는 BottomSheet의 커스텀 dim 오버레이 구현과 MY_FEED 탭으로의 네비게이션 수정이 이루어졌으며, 이는 '업로드'와 관련된 기능 개선입니다.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch qa/upload

---

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

[coderabbitai]

🧹 Nitpick comments (1)

core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/BottomSheet.kt (1)

146-149: 드래그 핸들 색상을 테마 색상으로 변경하는 것을 고려해 주세요.

다른 색상들은 BuyOrNotTheme.colors를 사용하고 있지만, 드래그 핸들은 하드코딩된 Color(0xFFD9D9D9)를 사용합니다. 일관성과 유지보수성을 위해 적절한 테마 색상(예: gray300 또는 새로운 디자인 토큰)을 사용하는 것이 좋습니다.

♻️ 테마 색상 사용 제안

                         .background(
-                            color = Color(0xFFD9D9D9),
+                            color = BuyOrNotTheme.colors.gray300,
                             shape = RoundedCornerShape(18.dp),
                         ),

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/BottomSheet.kt`
around lines 146 - 149, Replace the hardcoded drag-handle color
Color(0xFFD9D9D9) in the BottomSheet composable's background modifier with the
theme color from BuyOrNotTheme (e.g., BuyOrNotTheme.colors.gray300 or the
appropriate design token); locate the .background(...) call that currently sets
the handle shape and swap the literal Color(...) for the theme reference so the
handle follows the app's color system and stays consistent with other usages of
BuyOrNotTheme.colors.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In
`@core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/BottomSheet.kt`:
- Around line 146-149: Replace the hardcoded drag-handle color Color(0xFFD9D9D9)
in the BottomSheet composable's background modifier with the theme color from
BuyOrNotTheme (e.g., BuyOrNotTheme.colors.gray300 or the appropriate design
token); locate the .background(...) call that currently sets the handle shape
and swap the literal Color(...) for the theme reference so the handle follows
the app's color system and stays consistent with other usages of
BuyOrNotTheme.colors.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 5b0eb149-2d7c-4d53-9fda-7fe01689d6ad

📥 Commits

Reviewing files that changed from the base of the PR and between 82a4dc9 and 0c75bf8.

📒 Files selected for processing (2)

• core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/BottomSheet.kt
• feature/home/src/main/java/com/sseotdabwa/buyornot/feature/home/navigation/HomeNavigation.kt