Conversation
📝 Walkthrough개요디자인 시스템 텍스트 필드 컴포넌트를 재구성하는 변경사항입니다. 새로운 변경 사항
예상 코드 리뷰 소요시간🎯 3 (Moderate) | ⏱️ ~25 분 관련 가능성이 있는 PR
제안 레이블
제안 검토자
축하 시
🚥 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
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
app/src/main/java/com/flint/presentation/collectioncreate/CollectionCreateScreen.kt (1)
145-157: 제목 필드에 명시적 높이 설정 필요.현재 제목 필드는
modifier = Modifier.fillMaxWidth()로 설정되어 있지만, 높이가 지정되지 않았습니다.CollectionInputTextField컴포넌트의 Preview와 설명 필드의 패턴을 보면, 단일 행 입력 필드는.height(40.dp)를 사용하고 있습니다. 설명 필드는.heightIn(min = 104.dp)를 적용하고 있으므로, 제목 필드도.height(40.dp)를 추가하여 일관된 높이를 유지해야 합니다.
🧹 Nitpick comments (1)
app/src/main/java/com/flint/core/designsystem/component/textfield/CollectionInputTextField.kt (1)
39-62: Column 내부 return 동작 확인 필요.Line 55의
return은 Column 블록이 아닌 전체 composable 함수를 종료합니다. 현재 로직상 의도된 동작이지만, 향후 Column 아래에 다른 컴포넌트가 추가될 경우 예상치 못한 동작이 발생할 수 있습니다.또한,
isShowLengthTitle = true이고maxLength = Int.MAX_VALUE일 때 "0/2147483647"과 같이 표시됩니다. 이 경우를 처리하는 것이 좋을 수 있습니다.♻️ 명시적인 조건 처리 제안
- if (!isShowLengthTitle) return - - Text( - text = "${value.graphemeLength}/$maxLength", - style = FlintTheme.typography.caption1M12, - color = FlintTheme.colors.white, - ) + if (isShowLengthTitle && maxLength != Int.MAX_VALUE) { + Text( + text = "${value.graphemeLength}/$maxLength", + style = FlintTheme.typography.caption1M12, + color = FlintTheme.colors.white, + ) + }
📮 관련 이슈
📌 작업 내용
📸 스크린샷
TF_AS-IS.mp4
TF_TO-BE.mp4
2026-01-23.8.56.34.mov
🫛 To. 리뷰어
CollectionInputTextField프리뷰에 모든 경우의 수 넣어두었습니다.Summary by CodeRabbit
릴리스 노트
새로운 기능
개선 사항
✏️ Tip: You can customize this high-level summary in your review settings.