[Feat] ExploreScreen(탐색 화면)#120
Conversation
- VerticalPager를 활용한 탐색 화면 레이아웃 구현 - 개별 콘텐츠를 보여주는 ExplorePageItem 컴포넌트 구현 (포스터, 제목, 설명, 이동 버튼) - 모든 콘텐츠 확인 후 표시되는 ExploreEndPage 컴포넌트 구현 - FlintLogoTopAppbar 및 배경 그라데이션 적용 - 각 컴포넌트 및 전체 화면에 대한 Preview 코드 추가
|
Caution Review failedFailed to post review comments 📝 WalkthroughWalkthrough새로운 컬렉션 저장 관련 Compose UI 컴포넌트(FlintSaveDoneButton, FlintSaveNoneButton)와 디자인 시스템 확장, ContentModel 데이터 모델 확대를 추가합니다. CollectionDetailScreen과 ExploreScreen을 매개변수 기반 구성으로 재구성하였습니다. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~50분 Possibly related PRs
Suggested labels
Suggested reviewers
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 |
kimjw2003
left a comment
There was a problem hiding this comment.
코리 한번만 확인부탁드립니다! 수고하셨습니다!
| ExploreScreen( | ||
| modifier = Modifier.padding(paddingValues), | ||
| ) | ||
| // ExploreScreen( |
There was a problem hiding this comment.
p2
데이터는 나중에 넣더라도 PR에는 screen 호출이 되어있으면 좋겠습니다!
| if (pagerState.currentPage < pageCount) { | ||
| background(FlintTheme.colors.background) | ||
| } else { | ||
| background( | ||
| FlintTheme.colors.gradient900, | ||
| ) | ||
| } |
There was a problem hiding this comment.
p3
background(if(pagerState.currentPage < pageCount) FlintTheme.colors.background else FlintTheme.colors.gradient900)
이렇게도 정리할 수 있을거 같습니다!
| val pageCount: Int = contents.size | ||
| val pagerState: PagerState = rememberPagerState(pageCount = { pageCount + 1 }) |
There was a problem hiding this comment.
p3
Explore 화면에서 페이지를 넘기다가 Home이나 Profile같은 다른 탭으로 이동 후 다시 Explore로 돌아왔을때 보고 있던 페이지가 유지 되어야 한다고 기획에서 말씀주셨는데
혹시 이 형태로도 해당 상황에서 문제가 일어나지는 않을까 궁금합니다!
그렇게 생각한 근거로는
remember로 선언을 해놓으면 리컴포지션 상황에서는 재생성을 막을 수 있지만, screen자체가 재생성 되는 경우에는 아마 pagerState또한 재생성되며 해당 내용들이 초기화 되지 않을까 우려됩니다. 확인 한번 부탁드립니다!
There was a problem hiding this comment.
뷰모델 로직 작성하면서 확인해주시면 좋을 것 같아요!
| val pageCount: Int = contents.size | ||
| val pagerState: PagerState = rememberPagerState(pageCount = { pageCount + 1 }) |
There was a problem hiding this comment.
뷰모델 로직 작성하면서 확인해주시면 좋을 것 같아요!
| Modifier | ||
| .padding(bottom = 16.dp) | ||
| .fillMaxWidth() | ||
| .defaultMinSize(minHeight = 48.dp), |
📮 관련 이슈
📌 작업 내용
ExploreScreen(탐색 화면)📸 스크린샷
Screen_recording_20260116_143939.mp4
Summary by CodeRabbit
릴리스 노트
새 기능
개선 사항
✏️ Tip: You can customize this high-level summary in your review settings.