Conversation
- 기존 모달용 에셋 -> ic_modal_warning으로 수정
- 전역에서 사용할 수 있는 BitnagilToast 를 관리하는 GlobalBitnagilToast 싱글톤 객체 추가 - MainActivity에 BitnagilToastContainer 추가
- 루틴 완료/미완료 처리 실패 시 토스트 메시지 표시 - 감정 구슬 중복 등록 시 토스트 메시지 표시
Walkthrough앱 전체에서 사용할 수 있는 커스텀 토스트 메시지 시스템이 도입되었습니다. 새로운 디자인의 토스트 컴포넌트와 상태 관리, 글로벌 토스트 싱글턴, 그리고 각 화면 및 ViewModel에서 토스트 호출 로직이 추가 및 통합되었습니다. 기존 경고 아이콘 리소스도 개선 및 교체되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant MainScreen
participant HomeViewModel
participant GlobalBitnagilToast
participant BitnagilToastContainer
User->>MainScreen: 버튼 클릭/액션 발생
MainScreen->>HomeViewModel: HomeIntent 전달
HomeViewModel->>HomeViewModel: 상태/사이드이펙트 처리
HomeViewModel-->>MainScreen: HomeSideEffect(ShowToast 등)
MainScreen->>GlobalBitnagilToast: show()/showWarning()
GlobalBitnagilToast->>BitnagilToastContainer: 상태 업데이트
BitnagilToastContainer-->>User: 토스트 메시지 애니메이션 표시
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Assessment against linked issues
Poem
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (6)
core/designsystem/src/main/res/drawable/ic_modal_warning.xml (1)
9-16: 하드코딩된 색상 값 검토 필요색상 값이 하드코딩되어 있습니다 (
#C2C4C8,#ffffff). 다크 모드나 테마 변경을 고려하여BitnagilTheme.colors의 색상을 사용하는 것을 권장합니다.다음과 같이 색상 리소스를 정의하여 사용하는 것을 고려해보세요:
<path android:fillColor="?attr/modalWarningBackground" android:pathData="..." /> <path android:fillColor="?attr/modalWarningForeground" android:pathData="..." />core/designsystem/src/main/res/drawable/ic_warning.xml (1)
1-12: 경고 아이콘 단순화 및 크기 조정 적절아이콘을 55dp에서 24dp로 축소하고 디자인을 단순화한 것이 좋은 개선입니다. 표준적인 아이콘 크기를 사용하고 더 범용적인 디자인이 되었습니다.
하드코딩된 색상(
#FF6868) 대신 테마 색상 사용을 고려해보세요:android:fillColor="?attr/warningColor"presentation/src/main/java/com/threegap/bitnagil/presentation/home/model/HomeSideEffect.kt (1)
6-7: 토스트 메시지 구현이 명확하고 적절합니다.두 개의 토스트 side effect가 잘 구분되어 있습니다. 향후 확장성을 고려한다면 아이콘을 옵셔널 파라미터로 하는 단일 클래스도 고려해볼 수 있지만, 현재 구현은 명확하고 MVI 패턴에 잘 부합합니다.
presentation/src/main/java/com/threegap/bitnagil/presentation/common/toast/GlobalBitnagilToast.kt (1)
14-16: WeakReference null 처리에 대한 로깅 고려.현재
_toastStateRef?.get()이 null을 반환하면 조용히 무시되는데, 디버깅을 위해 로그를 추가하는 것을 고려해보세요.fun show(text: String, icon: Int? = null) { - _toastStateRef?.get()?.show(text, icon) + _toastStateRef?.get()?.show(text, icon) + ?: Log.w("GlobalBitnagilToast", "ToastState is not initialized or has been garbage collected") }core/designsystem/src/main/java/com/threegap/bitnagil/designsystem/component/atom/BitnagilToastMessage.kt (2)
77-83: LaunchedEffect에서 중복 조건 체크를 제거하세요.Line 77에서
state.isVisible을 이미 체크했는데 Line 79에서 다시 체크하는 것은 불필요합니다.LaunchedEffect(state.toastId) { - if (state.isVisible) { - delay(duration) - state.hide() - } + delay(duration) + state.hide() }
55-55: 아이콘 색상이 하드코딩되어 있습니다.경고 아이콘에
BitnagilTheme.colors.error색상이 하드코딩되어 있는데, 아이콘 타입에 따라 다른 색상을 사용할 수 있도록 개선하는 것을 고려해보세요.BitnagilIcon( id = id, - tint = BitnagilTheme.colors.error, + tint = when (id) { + R.drawable.ic_warning -> BitnagilTheme.colors.error + R.drawable.ic_check -> BitnagilTheme.colors.success + else -> BitnagilTheme.colors.white + }, modifier = Modifier .padding(end = 4.dp) .size(24.dp), )
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (11)
app/src/main/java/com/threegap/bitnagil/MainActivity.kt(2 hunks)app/src/main/java/com/threegap/bitnagil/navigation/home/HomeNavHost.kt(3 hunks)core/designsystem/src/main/java/com/threegap/bitnagil/designsystem/component/atom/BitnagilToastMessage.kt(1 hunks)core/designsystem/src/main/res/drawable/ic_modal_warning.xml(1 hunks)core/designsystem/src/main/res/drawable/ic_warning.xml(1 hunks)presentation/src/main/java/com/threegap/bitnagil/presentation/common/toast/GlobalBitnagilToast.kt(1 hunks)presentation/src/main/java/com/threegap/bitnagil/presentation/home/HomeScreen.kt(5 hunks)presentation/src/main/java/com/threegap/bitnagil/presentation/home/HomeViewModel.kt(2 hunks)presentation/src/main/java/com/threegap/bitnagil/presentation/home/model/HomeIntent.kt(1 hunks)presentation/src/main/java/com/threegap/bitnagil/presentation/home/model/HomeSideEffect.kt(1 hunks)presentation/src/main/java/com/threegap/bitnagil/presentation/setting/component/block/ConfirmDialog.kt(1 hunks)
🧰 Additional context used
🧠 Learnings (7)
app/src/main/java/com/threegap/bitnagil/navigation/home/HomeNavHost.kt (1)
Learnt from: l5x5l
PR: #38
File: presentation/src/main/java/com/threegap/bitnagil/presentation/writeroutine/component/atom/textbutton/TextButton.kt:30-35
Timestamp: 2025-07-21T10:38:49.104Z
Learning: presentation/src/main/java/com/threegap/bitnagil/presentation/writeroutine/component/atom/textbutton/TextButton.kt의 TextButton 컴포넌트는 임시로 구현된 컴포넌트로, 디자인 시스템 구현시 대체 예정입니다.
presentation/src/main/java/com/threegap/bitnagil/presentation/home/model/HomeIntent.kt (1)
Learnt from: l5x5l
PR: #41
File: presentation/src/main/java/com/threegap/bitnagil/presentation/mypage/model/MyPageIntent.kt:6-6
Timestamp: 2025-07-23T13:32:26.263Z
Learning: In the Bitnagil Android project's MVI architecture, Intent classes like LoadMyPageSuccess are named to represent successful API response results that carry loaded data, not just user actions. This naming convention is used for future API integration where the intent will be triggered when my page data loading succeeds from the server.
presentation/src/main/java/com/threegap/bitnagil/presentation/home/HomeScreen.kt (2)
Learnt from: l5x5l
PR: #41
File: presentation/src/main/java/com/threegap/bitnagil/presentation/mypage/model/MyPageIntent.kt:6-6
Timestamp: 2025-07-23T13:32:26.263Z
Learning: In the Bitnagil Android project's MVI architecture, Intent classes like LoadMyPageSuccess are named to represent successful API response results that carry loaded data, not just user actions. This naming convention is used for future API integration where the intent will be triggered when my page data loading succeeds from the server.
Learnt from: l5x5l
PR: #38
File: presentation/src/main/java/com/threegap/bitnagil/presentation/writeroutine/component/atom/textbutton/TextButton.kt:30-35
Timestamp: 2025-07-21T10:38:49.104Z
Learning: presentation/src/main/java/com/threegap/bitnagil/presentation/writeroutine/component/atom/textbutton/TextButton.kt의 TextButton 컴포넌트는 임시로 구현된 컴포넌트로, 디자인 시스템 구현시 대체 예정입니다.
presentation/src/main/java/com/threegap/bitnagil/presentation/home/HomeViewModel.kt (1)
Learnt from: l5x5l
PR: #41
File: presentation/src/main/java/com/threegap/bitnagil/presentation/mypage/model/MyPageIntent.kt:6-6
Timestamp: 2025-07-23T13:32:26.263Z
Learning: In the Bitnagil Android project's MVI architecture, Intent classes like LoadMyPageSuccess are named to represent successful API response results that carry loaded data, not just user actions. This naming convention is used for future API integration where the intent will be triggered when my page data loading succeeds from the server.
app/src/main/java/com/threegap/bitnagil/MainActivity.kt (1)
Learnt from: l5x5l
PR: #38
File: presentation/src/main/java/com/threegap/bitnagil/presentation/writeroutine/component/atom/textbutton/TextButton.kt:30-35
Timestamp: 2025-07-21T10:38:49.104Z
Learning: presentation/src/main/java/com/threegap/bitnagil/presentation/writeroutine/component/atom/textbutton/TextButton.kt의 TextButton 컴포넌트는 임시로 구현된 컴포넌트로, 디자인 시스템 구현시 대체 예정입니다.
presentation/src/main/java/com/threegap/bitnagil/presentation/home/model/HomeSideEffect.kt (1)
Learnt from: l5x5l
PR: #41
File: presentation/src/main/java/com/threegap/bitnagil/presentation/mypage/model/MyPageIntent.kt:6-6
Timestamp: 2025-07-23T13:32:26.263Z
Learning: In the Bitnagil Android project's MVI architecture, Intent classes like LoadMyPageSuccess are named to represent successful API response results that carry loaded data, not just user actions. This naming convention is used for future API integration where the intent will be triggered when my page data loading succeeds from the server.
core/designsystem/src/main/java/com/threegap/bitnagil/designsystem/component/atom/BitnagilToastMessage.kt (1)
Learnt from: l5x5l
PR: #38
File: presentation/src/main/java/com/threegap/bitnagil/presentation/writeroutine/component/atom/textbutton/TextButton.kt:30-35
Timestamp: 2025-07-21T10:38:49.104Z
Learning: presentation/src/main/java/com/threegap/bitnagil/presentation/writeroutine/component/atom/textbutton/TextButton.kt의 TextButton 컴포넌트는 임시로 구현된 컴포넌트로, 디자인 시스템 구현시 대체 예정입니다.
🧬 Code Graph Analysis (3)
presentation/src/main/java/com/threegap/bitnagil/presentation/home/HomeViewModel.kt (1)
presentation/src/main/java/com/threegap/bitnagil/presentation/common/mviviewmodel/MviViewModel.kt (2)
sendSideEffect(23-23)sendIntent(30-37)
app/src/main/java/com/threegap/bitnagil/MainActivity.kt (2)
core/designsystem/src/main/java/com/threegap/bitnagil/designsystem/component/atom/BitnagilToastMessage.kt (2)
rememberBitnagilToast(136-137)BitnagilToastContainer(71-97)app/src/main/java/com/threegap/bitnagil/MainScreen.kt (1)
MainScreen(9-22)
core/designsystem/src/main/java/com/threegap/bitnagil/designsystem/component/atom/BitnagilToastMessage.kt (1)
core/designsystem/src/main/java/com/threegap/bitnagil/designsystem/component/atom/BitnagilIcon.kt (1)
BitnagilIcon(21-33)
⏰ 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: build
🔇 Additional comments (31)
core/designsystem/src/main/res/drawable/ic_modal_warning.xml (1)
2-5: 아이콘 크기 적절성 확인55dp는 일반적인 아이콘 크기보다 큽니다. 모달 다이얼로그용 특수 아이콘이라면 적절하지만, 다른 UI 컨텍스트에서 재사용성을 고려하여 크기가 적절한지 확인해주세요.
presentation/src/main/java/com/threegap/bitnagil/presentation/setting/component/block/ConfirmDialog.kt (1)
56-56: LGTM! 모달 전용 경고 아이콘으로 적절한 업데이트새로 추가된
ic_modal_warning리소스로의 변경이 적절합니다. 모달 다이얼로그 컨텍스트에 맞는 더 상세한 아이콘을 사용하게 되었습니다.app/src/main/java/com/threegap/bitnagil/navigation/home/HomeNavHost.kt (3)
28-28: LGTM! 글로벌 토스트 시스템 통합
GlobalBitnagilToast임포트가 적절하게 추가되었습니다.
118-118: 패딩 모디파이어 통합으로 코드 정리여러 개의
.padding()호출을 하나로 통합한 것이 좋은 개선입니다.
134-134: 사용자 경험 개선을 위한 토스트 메시지 추가뒤로가기 버튼 두 번 누르기 패턴에 토스트 메시지를 추가하여 사용자에게 명확한 피드백을 제공하는 것이 좋은 UX 개선입니다.
app/src/main/java/com/threegap/bitnagil/MainActivity.kt (2)
26-30: 글로벌 토스트 상태 초기화 구현 적절
rememberBitnagilToast()로 상태를 생성하고LaunchedEffect로GlobalBitnagilToast를 초기화하는 패턴이 Compose 모범 사례를 따르고 있습니다.
32-43: 토스트 컨테이너 레이아웃 구성 검토
Box레이아웃으로 토스트 컨테이너를 오버레이하는 구조가 적절합니다. 하단 100dp 패딩은 bottom navigation과의 충돌을 방지하기에 적절한 값으로 보입니다.토스트 메시지가 다른 UI 요소(예: 플로팅 액션 버튼)와 겹치지 않는지 확인해주세요.
presentation/src/main/java/com/threegap/bitnagil/presentation/common/toast/GlobalBitnagilToast.kt (2)
7-21: 잘 설계된 글로벌 토스트 싱글턴입니다.WeakReference를 사용하여 메모리 누수를 방지하고, null-safe 접근 패턴을 적용한 것이 좋습니다. 편의 메서드들도 API를 깔끔하게 만들어 줍니다.
7-21: 전역 토스트 시스템 구현이 우수합니다.WeakReference를 사용하여 메모리 누수를 방지하고, 편의 메서드들을 제공하는 설계가 좋습니다.
presentation/src/main/java/com/threegap/bitnagil/presentation/home/HomeScreen.kt (6)
29-30: 적절한 import 추가입니다.새로운 기능에 필요한 import들이 올바르게 추가되었습니다.
Also applies to: 39-39
54-76: MVI 아키텍처를 잘 따른 사이드 이펙트 처리입니다.각 사이드 이펙트 타입에 따라 적절한 액션을 수행하고, 글로벌 토스트 시스템을 올바르게 활용하고 있습니다.
95-95: 네비게이션 로직의 중앙집중화가 잘 되었습니다.직접적인 네비게이션 콜백 호출을 Intent 전송으로 변경하여 관심사 분리가 개선되었습니다. MVI 아키텍처에 잘 부합합니다.
Also applies to: 147-152
29-30: 필요한 임포트가 정확히 추가되었습니다.새로운 토스트 시스템과 사이드 이펙트 처리를 위한 임포트가 적절합니다.
54-76: MVI 패턴을 잘 적용한 사이드 이펙트 처리입니다.UI에서 직접 네비게이션을 호출하지 않고 ViewModel의 사이드 이펙트를 통해 처리하는 방식이 관심사 분리 원칙에 잘 맞습니다. 토스트 표시도 전역 시스템을 통해 일관되게 처리되고 있습니다.
147-152: 직접 네비게이션 호출을 인텐트 전송으로 개선했습니다.기존에 콜백을 직접 호출하던 방식에서 ViewModel로 인텐트를 전송하는 방식으로 변경하여 MVI 아키텍처를 더 잘 따르게 되었습니다.
presentation/src/main/java/com/threegap/bitnagil/presentation/home/model/HomeIntent.kt (2)
23-26: 새로운 Intent들이 잘 설계되었습니다.네이밍 컨벤션이 일관성 있고, 데이터가 필요한 경우 data class, 단순한 액션의 경우 data object를 적절히 사용했습니다. MVI 아키텍처에 잘 부합합니다.
23-26: 새로운 인텐트들이 적절하게 정의되었습니다.네비게이션과 실패 케이스를 위한 인텐트들이 MVI 패턴에 맞게 잘 정의되었습니다. 특히
RoutineToggleCompletionFailure처럼 실패 상황을 명시적으로 처리하는 것이 좋습니다.presentation/src/main/java/com/threegap/bitnagil/presentation/home/HomeViewModel.kt (6)
204-226: 새로운 Intent 처리 로직이 잘 구현되었습니다.OnRegisterEmotionClick의 조건부 로직이 적절하고, 모든 사이드 이펙트 발생이 올바르게 구현되었습니다. 상태 변경이 없는 경우 null을 반환하는 것도 적절합니다.
477-477: 에러 처리가 개선되었습니다.루틴 동기화 실패 시 사용자에게 피드백을 제공하는 Intent를 추가하여 사용자 경험이 향상되었습니다.
204-211: 감정 등록 클릭 처리 로직이 우수합니다.이미 감정이 등록된 경우와 아닌 경우를 구분하여 처리하는 조건부 로직이 잘 구현되었습니다. 사용자 경험을 고려한 좋은 설계입니다.
213-216: 루틴 등록 네비게이션이 올바르게 구현되었습니다.사이드 이펙트를 통한 네비게이션 처리가 MVI 패턴에 맞게 잘 구현되었습니다.
218-221: 실패 케이스 처리가 개선되었습니다.루틴 토글 실패 시 사용자에게 명확한 피드백을 제공하는 것이 좋습니다.
477-477: 실패 처리에 토스트 알림이 추가되었습니다.동기화 실패 시 사용자에게 피드백을 제공하여 UX가 개선되었습니다.
core/designsystem/src/main/java/com/threegap/bitnagil/designsystem/component/atom/BitnagilToastMessage.kt (6)
34-69: 토스트 메시지 컴포넌트가 잘 설계되었습니다.디자인 시스템을 일관성 있게 사용하고, 옵셔널 아이콘 지원이 깔끔하게 구현되었습니다.
71-97: 토스트 컨테이너 구현이 훌륭합니다.애니메이션과 자동 숨기기 기능이 잘 구현되었고, toastId를 키로 사용하여 재구성을 적절히 트리거합니다.
99-134: 상태 관리가 잘 구현되었습니다.적절한 캡슐화와 Compose 반응성을 위한 mutableStateOf 사용이 올바릅니다. toastId를 통한 재구성 트리거링도 적절합니다.
122-125: 중복 방지 로직이 잘 구현되었습니다.500ms 윈도우와 텍스트/아이콘 비교를 통한 중복 방지가 적절합니다. 다만 빠른 연속 호출 시나리오에서의 동작을 고려해보세요.
34-69: 토스트 UI 컴포넌트가 잘 설계되었습니다.아이콘과 텍스트를 함께 표시할 수 있는 유연한 구조와 적절한 스타일링이 적용되었습니다.
99-134: 토스트 상태 관리가 잘 구현되었습니다.중복 토스트 방지 로직과 상태 관리가 깔끔하게 구현되었습니다. toastId를 사용한 리컴포지션 트리거링도 좋은 접근입니다.
presentation/src/main/java/com/threegap/bitnagil/presentation/home/model/HomeSideEffect.kt (2)
8-10: 네비게이션 side effect 구조가 일관되고 적절합니다.파라미터가 필요한 네비게이션은 data class로, 단순 네비게이션은 data object로 구현하여 일관된 패턴을 유지하고 있습니다. "NavigateTo" 접두사를 사용한 명명 규칙도 명확합니다.
5-11: MVI 아키텍처 패턴을 잘 따르는 구조입니다.sealed interface를 사용하여 타입 안전성을 보장하고, MviSideEffect를 확장하여 기존 아키텍처와 일관성을 유지합니다. 토스트 메시지와 네비게이션 기능을 명확히 분리한 설계가 우수합니다.
...ntation/src/main/java/com/threegap/bitnagil/presentation/common/toast/GlobalBitnagilToast.kt
Show resolved
Hide resolved
2 participants
[ PR Content ]
앱 전역에서 사용 가능한 커스텀 토스트 메시지를 구현했습니다.
Related issue
Screenshot 📸
default.mp4
Work Description
To Reviewers 📢
이 문제를
BitnagilToastState의 toastId 증가 방식과LaunchedEffect(state.toastId)를 통해 해결하여 새로운 메시지 표시 시마다 타이머가 2초로 정확히 리셋되도록 하는 방식으로 해결했습니다.Summary by CodeRabbit
신규 기능
버그 수정
디자인/스타일
ic_modal_warning)이 추가되고, 기존 경고 아이콘(ic_warning)이 더 간결하게 리디자인되었습니다.리팩터링