[Feat] FlintProgressBar 구현했어요 ~!#80
Conversation
📝 WalkthroughWalkthrough새로운 Jetpack Compose UI 컴포넌트 Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
app/src/main/java/com/flint/core/designsystem/component/progressbar/FlintProgressBar.kt (2)
19-21:progress값 범위 검증 추가를 권장합니다.
fillMaxWidth(fraction)은 0~1 범위의 값을 기대합니다. 범위를 벗어난 값이 전달될 경우 예상치 못한 동작이 발생할 수 있으므로,coerceIn을 사용하여 입력값을 명시적으로 제한하는 것이 좋습니다.♻️ 제안하는 수정안
@Composable fun FlintProgressBar( progress: Float, modifier: Modifier = Modifier, ) { + val clampedProgress = progress.coerceIn(0f, 1f) Box( modifier = modifier .height(4.dp) .clip(CircleShape) .background(FlintTheme.colors.gray600), ) { Box( modifier = Modifier - .fillMaxWidth(progress) + .fillMaxWidth(clampedProgress) .fillMaxHeight() .clip(CircleShape) .background(FlintTheme.colors.secondary400), ) } }
23-38: 접근성(Accessibility) 개선을 고려해 주세요.스크린 리더 사용자를 위해
semanticsmodifier를 추가하면 더 좋은 사용자 경험을 제공할 수 있습니다. 현재 구현은 기능적으로 정상 동작하지만, 접근성 요구사항이 있는 프로젝트라면 고려해 보세요.♻️ 접근성 개선 예시
import androidx.compose.foundation.progressSemantics // Box에 semantics 추가 Box( modifier = modifier .progressSemantics(progress) .height(4.dp) .clip(CircleShape) .background(FlintTheme.colors.gray600), ) { // ... }
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
app/src/main/java/com/flint/core/designsystem/component/progressbar/FlintProgressBar.kt
🧰 Additional context used
🧬 Code graph analysis (1)
app/src/main/java/com/flint/core/designsystem/component/progressbar/FlintProgressBar.kt (1)
app/src/main/java/com/flint/core/designsystem/theme/Theme.kt (1)
FlintTheme(8-16)
🔇 Additional comments (3)
app/src/main/java/com/flint/core/designsystem/component/progressbar/FlintProgressBar.kt (3)
1-16: LGTM!import 구성이 적절하며, 필요한 Compose 컴포넌트와 테마 관련 의존성이 잘 정리되어 있습니다.
41-49: LGTM!
FlintTheme으로 올바르게 감싸고,PreviewParameter를 사용하여 다양한 진행 상태를 미리볼 수 있도록 잘 구현되었습니다.
51-53: LGTM!경계값(0%, 100%)과 중간값들을 포함하여 다양한 진행 상태를 테스트할 수 있도록 잘 구성되었습니다.
|
피그마 확인해보니까 온보딩에서 쓰이는 프로그래스바랑 컬렉션 상세보기에 쓰이는 프로그래스바 비활성 부분 Radius가 다르네요! |
📮 관련 이슈
📌 작업 내용
FlintProgressBar구현📸 스크린샷
🫛 To. 리뷰어
피자 열판 쏘겠습니다^^
허리피자.
가슴피자.
얼굴피자.
어깨피자.
다리피자.
주름피자.
형편피자.
인생피자.
팔자피자.
웃음피자.
2026년 새해 건강과 행복이 함께 하시길 기원 합니다.
🍂🍁~^^~* 🍃🌾
세상 살아 가면서
천만번 해도 부족한 말.
😊감사 합니다😊
🌻사랑 합니다🌻
🍀고맙 습니다🍀
🌹건강 하세요🌹
🌿행복 하세요🌿
Summary by CodeRabbit
릴리스 노트
✏️ Tip: You can customize this high-level summary in your review settings.