[Fix] 바텀 내비게이션 UI 수정 #165

[ProtossManse]

🔗 관련 이슈

• Close [Fix] 바텀 내비게이션 UI 수정 #165

📙 작업 설명

• 바텀 내비게이션 바 background 추가
• MainActivity의 Scaffold -> containerColor 추가

Summary by CodeRabbit

• 스타일 개선
  • 메인 화면 컨테이너 배경색을 앱 테마에 맞게 조정했습니다.
  • 하단 네비게이션 바의 배경색을 테마 색상으로 업데이트하여 전체 UI 일관성을 개선했습니다.

[coderabbitai]

Walkthrough

메인 액티비티와 바텀 내비게이션 바의 배경색을 테마 색상과 일치하도록 조정했습니다. Scaffold 컨테이너 색상을 백그라운드 색상으로 설정하고, 바텀 바 배경을 흰색으로 통일하여 Figma 디자인과의 정렬을 개선했습니다.

Changes

Cohort / File(s)	변경 사항
UI 스타일링 업데이트 app/src/main/java/com/konkuk/medicarecall/MainActivity.kt, app/src/main/java/com/konkuk/medicarecall/ui/navigation/component/MainBottomBar.kt	테마 색상을 적용하여 컨테이너와 배경색 정렬. Scaffold에 containerColor 파라미터 추가 및 MainBottomBar의 Box 배경을 테마 색상으로 설정

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• 두 파일 모두 단순한 색상 속성 추가만 포함
• 로직 변경 없이 순수 UI 스타일링만 적용
• 기존 코드 구조에 영향 없음

Suggested labels

fix

Suggested reviewers

• alswlekk
• ikseong00

Poem

🎨 색상 맞춘 UI 반짝반짝 ✨
피그마와 코드가 이제 친구
바텀 바도 우아하게 배경 입고
머터리얼은 안녕, 테마 색이 주인공! 🌈

Pre-merge checks and finishing touches

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목 '[Fix] 바텀 내비게이션 UI 수정 #165'는 변경사항의 주요 내용을 명확하게 나타냅니다. MainActivity의 Scaffold에 containerColor를 추가하고 MainBottomBar에 배경색을 추가한 것으로, 이는 바텀 네비게이션 UI 수정이라는 제목과 정확히 일치합니다.
Linked Issues check	✅ Passed	PR의 변경사항이 Issue #165의 요구사항을 충족합니다. Issue #165에서는 Material 기본 보라색 대신 디자인에 맞는 컨테이너 색상으로 변경할 것을 요청했으며, 본 PR은 MainBottomBar에 MediCareCallTheme.colors.white 배경을 추가하고 MainActivity의 Scaffold에 containerColor를 MediCareCallTheme.colors.bg로 설정하여 이 요구사항을 이행했습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 Issue #165의 범위 내에 있습니다. MainActivity.kt와 MainBottomBar.kt의 변경은 바텀 네비게이션 UI 수정과 직접적으로 관련된 것이며, 추가적인 비즈니스 로직 변경이나 다른 기능 수정은 없습니다.

✨ 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/bottom-navigation-ui-#165

---

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]

Actionable comments posted: 0

🧹 Nitpick comments (1)

app/src/main/java/com/konkuk/medicarecall/MainActivity.kt (1)

73-74: 배경색 설정이 중복될 수 있습니다.

73번 라인의 .background() 모디파이어와 74번 라인의 containerColor 파라미터가 동일한 색상을 설정하고 있네요. Scaffold의 containerColor 파라미터가 배경색을 설정하는 표준 방법이므로, 73번 라인의 .background() 모디파이어는 불필요할 수 있습니다.

다음과 같이 수정하면 더 간결해집니다:

 Scaffold(
-    modifier = Modifier.background(MediCareCallTheme.colors.bg),
     containerColor = MediCareCallTheme.colors.bg,
     contentWindowInsets = WindowInsets.systemBars.only(WindowInsetsSides.Horizontal),

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d9a6dea and 7410c9f.

📒 Files selected for processing (2)

• app/src/main/java/com/konkuk/medicarecall/MainActivity.kt (1 hunks)
• app/src/main/java/com/konkuk/medicarecall/ui/navigation/component/MainBottomBar.kt (2 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: ci-build

🔇 Additional comments (1)

app/src/main/java/com/konkuk/medicarecall/ui/navigation/component/MainBottomBar.kt (1)

53-58: 바텀 내비게이션 배경 추가가 잘 구현되었습니다!

흰색 배경에 회색 테두리를 추가하여 Figma 디자인과 일치시켰네요. 모디파이어 순서도 올바르게 설정되어 있습니다 (background → border).

[alswlekk]

👍

[librawish808]

좋습니다