Conversation
Walkthrough온보딩 종료 화면의 LottieAnimation에 둥근 모서리 클리핑을 적용하고, 고정 높이 Spacer를 가중치 기반 Spacer로 교체했습니다. 관련 Modifier 및 RoundedCornerShape import가 추가되었으며, 공개 API 변경은 없습니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Assessment against linked issues
Assessment against linked issues: Out-of-scope changes(없음) Suggested reviewers
✨ 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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
app/src/main/java/com/spoony/spoony/presentation/auth/onboarding/OnboardingEndScreen.kt (1)
93-95: clip만 적용 시 크기 미정 가능성 — 안정적 레이아웃을 위해 크기 지정 권장라운드 클리핑만 있고 사이즈가 없어 기기/비율에 따라 표시 크기가 들쭉날쭉할 수 있습니다. 폭 기준으로 맞추고 클리핑을 적용하면 더 예측 가능한 UI가 됩니다.
- modifier = Modifier - .clip(RoundedCornerShape(8.dp)) + modifier = Modifier + .fillMaxWidth() + .clip(RoundedCornerShape(8.dp))필요하다면
aspectRatio()를 추가해 고정 비율을 유지하는 것도 고려해 주세요.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
app/src/main/java/com/spoony/spoony/presentation/auth/onboarding/OnboardingEndScreen.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: PR Checker
🔇 Additional comments (2)
app/src/main/java/com/spoony/spoony/presentation/auth/onboarding/OnboardingEndScreen.kt (2)
10-10: 임포트 추가 적절RoundedCornerShape/clip 임포트 추가는 둥근 모서리 클리핑 목적에 부합합니다.
Also applies to: 17-17
92-92: 무한 반복 재생 의도/수명 주기 확인 요청
LottieConstants.IterateForever가 의도라면 OK입니다. 다만 화면 전환 시(예:navigateToMap) 해당 컴포저블이 즉시 제거되어 애니메이션이 더 이상 재생되지 않는지, 혹은 백스택에 남아 불필요하게 재생되지 않는지 확인 부탁드립니다. 필요 시isPlaying을 화면 가시성에 바인딩하세요.
| ) | ||
|
|
||
| Spacer(modifier = Modifier.height(30.dp)) | ||
| Spacer(modifier = Modifier.weight(66f)) |
There was a problem hiding this comment.
Modifier.weight 임포트 누락으로 컴파일 실패
현재 androidx.compose.foundation.layout.weight 임포트가 없어 빌드가 실패합니다. 아래 임포트를 추가해 주세요. 또한 가중치 값은 비율만 의미하므로 66f/79f처럼 큰 값 대신 2f/3f 등 간결한 비율로 표현해도 동일하게 동작합니다.
@@
-import androidx.compose.foundation.layout.padding
+import androidx.compose.foundation.layout.padding
+import androidx.compose.foundation.layout.weightAlso applies to: 97-97
🤖 Prompt for AI Agents
In
app/src/main/java/com/spoony/spoony/presentation/auth/onboarding/OnboardingEndScreen.kt
around lines 88 and 97, the build fails because
androidx.compose.foundation.layout.weight is not imported and the Spacer weight
values use unnecessarily large absolute numbers; add the missing import for
Modifier.weight (androidx.compose.foundation.layout.weight) and replace the
large float weights (e.g., 66f and 13f) with simplified proportional values (for
example 2f and 1f or 2f/3f equivalents) so the layout behavior remains the same
and the file compiles.
There was a problem hiding this comment.
. 현재 코드에서 Spacer들이 Column 안에 있기 때문에 추가 import 없이도 정상적으로 컴파일됩니다.
제가 불필요한 import 오류를 지적한 점 죄송합니다. 코드는 현재 상태로 문제없이 동작할 것입니다.
🐰💭
╭─────╮
│ 실수 │
│했네요│
╰─────╯
4 participants
Related issue 🛠
Work Description ✏️
Screenshot 📸
Screen_Recording_20250907_064823.mp4
Screen_Recording_20250907_065359.mp4
Uncompleted Tasks 😅
To Reviewers 📢
로띠 파일이랑 간단한 디자인 수정사항만 수정되었습니다!
지금 디쌤들이 로띠 수정해주고 계셔서 잠시 draft 걸어두겠습니다! 수정된 로띠파일 받으면 draft 풀게요
Summary by CodeRabbit