[Feat] 프로필 화면 UI 구현#114
Conversation
📝 WalkthroughWalkthrough이 PR은 홈 및 프로필 화면 UI 컴포넌트를 리팩토링하여 새로운 모듈식 섹션들(CollectionSection, SavedContentsSection, ProfileKeywordSection, ProfileTopSection)을 도입하고, 기존 컴포넌트를 제거하며, 불변 리스트 타입으로 마이그레이션하고 테스트 데이터 모델을 추가합니다. Changes
Sequence Diagram(s)sequenceDiagram
participant ProfileScreen
participant LazyColumn
participant TopSection
participant KeywordSection
participant CollectionSection
participant SavedContentsSection
ProfileScreen->>LazyColumn: 렌더링 시작 (sections 반복)
LazyColumn->>TopSection: TopSection 렌더링
TopSection-->>LazyColumn: 프로필 헤더 표시
LazyColumn->>KeywordSection: KeywordSection 렌더링 (keywordList)
KeywordSection->>KeywordSection: rotateKeywordByRank 호출
KeywordSection-->>LazyColumn: 키워드 칩 + 그래프 표시
LazyColumn->>CollectionSection: CollectionSection 렌더링 (컬렉션 목록)
CollectionSection-->>LazyColumn: 컬렉션 항목 표시
LazyColumn->>SavedContentsSection: SavedContentsSection 렌더링 (콘텐츠 목록)
SavedContentsSection-->>LazyColumn: 저장한 콘텐츠 항목 표시
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (2)
app/src/main/java/com/flint/presentation/home/HomeScreen.kt (2)
30-46:navigateToCollectionList와navigateToCollectionCreate가 HomeScreen에 전달되지 않습니다.
HomeRoute에서 받은navigateToCollectionList와navigateToCollectionCreate콜백이HomeScreen에 전달되지 않아 FAB 클릭과 "전체보기" 기능이 동작하지 않습니다.🐛 수정 제안
fun HomeRoute( paddingValues: PaddingValues, navigateToCollectionList: () -> Unit, navigateToCollectionDetail: (collectionId: String) -> Unit, navigateToCollectionCreate: () -> Unit, ) { HomeScreen( onRecommendCollectionItemClick = { collectionId -> navigateToCollectionDetail(collectionId) }, onSavedContentItemClick = { contentId -> // TODO show OttListBottomSheet }, + onRecentCollectionAllClick = navigateToCollectionList, + navigateToCollectionCreate = navigateToCollectionCreate, modifier = Modifier.padding(paddingValues), ) }
154-154: 주석의emptyList()대신persistentListOf()를 사용하세요.
ImmutableList타입을 사용하고 있으므로, 빈 리스트를 전달할 때는emptyList()대신persistentListOf()를 사용해야 합니다.📝 수정 제안
- recentCollectionModelList = collectionModelList, // or 'emptyList()' + recentCollectionModelList = collectionModelList, // or 'persistentListOf()'
🤖 Fix all issues with AI agents
In `@app/src/main/java/com/flint/domain/model/PreferenceKeywordModel.kt`:
- Around line 17-32: rotateKeywordByRank currently zips topRanks and bottomRanks
which drops any unmatched elements when sizes differ (e.g., 4 or 5 items), so
change the assembly logic to interleave pairs from topRanks and bottomRanks and
then append any remaining elements from the longer list; specifically, in
rotateKeywordByRank after computing topRanks and bottomRanks use the zip result
plus the remainder of topRanks or bottomRanks (starting at the zip size) to
ensure no element is lost, preserving original rank order within each subgroup.
In
`@app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordSection.kt`:
- Around line 97-100: Calling keywordList.subList(0, MAX) can throw
IndexOutOfBoundsException when keywordList.size < MAX; update the call in
ProfileKeywordSection (where KeywordGraphLayout is invoked) to pass
keywordList.take(MAX) (and convert to immutable with toImmutableList() if an
immutable list is required), ensuring to import toImmutableList() from
kotlinx.collections.immutable when used.
🧹 Nitpick comments (12)
app/src/main/java/com/flint/domain/model/CollectionModel.kt (1)
14-46: FakeList 데이터에 고유 ID 사용을 권장합니다.Preview나 테스트용 더미 데이터로 적절하지만,
collectionId가 빈 문자열이고userId가 모두 0입니다.LazyColumn에서key로 사용할 경우 중복 키 문제가 발생할 수 있으므로, 각 항목에 고유한 ID 값을 설정하는 것이 좋습니다.💡 고유 ID 적용 예시
CollectionModel( - collectionId = "", + collectionId = "fake_1", collectionTitle = "컬렉션 제목", ... author = AuthorModel( - userId = 0, + userId = 1, ... ), ), CollectionModel( - collectionId = "", + collectionId = "fake_2", collectionTitle = "컬렉션 제목2", ... author = AuthorModel( - userId = 0, + userId = 2, ... ), ),app/src/main/java/com/flint/domain/model/ContentModel.kt (1)
13-62: FakeList 항목에 고유한 contentId 사용을 권장합니다.CollectionModel.FakeList와 동일하게 모든 항목의
contentId가 0으로 설정되어 있습니다. 리스트 렌더링 시key파라미터로 사용될 경우 성능 저하나 예상치 못한 동작이 발생할 수 있습니다.💡 고유 contentId 적용 예시
ContentModel( - contentId = 0, + contentId = 1, title = "드라마 제목", ... ), ContentModel( - contentId = 0, + contentId = 2, title = "드라마 제목2", ... ), ContentModel( - contentId = 0, + contentId = 3, title = "드라마 제목3", ... ), ContentModel( - contentId = 0, + contentId = 4, title = "드라마 제목4", ... ),app/src/main/java/com/flint/presentation/profile/component/ProfileTopSection.kt (2)
38-96: 불필요한 외부 Column 래퍼 제거를 권장합니다.외부
Column(Line 38)은 하나의Box자식만 포함하고 있어 불필요합니다. 또한modifier파라미터가 내부Box에 적용되어 있어, 외부에서 전달된 modifier가 예상과 다르게 동작할 수 있습니다.♻️ 리팩토링 제안
`@Composable` fun ProfileTopSection( userName: String, profileUrl: String, isFliner: Boolean, modifier: Modifier = Modifier, ) { - Column { - Box( - modifier = - modifier - .fillMaxWidth() - .height(260.dp) + Box( + modifier = + modifier + .fillMaxWidth() + .height(260.dp) ... - ) - } + ) }
49-52: 불필요한.copy(1f)호출 제거
Color(0xFF424BBD).copy(1f)에서.copy(1f)는 기본값(alpha = 1f)이므로 생략할 수 있습니다.💡 간소화 제안
val gradient = Brush.verticalGradient( - colors = listOf(Color(0xFF424BBD).copy(1f), Color(0xFF121212).copy(alpha = 0.04f)), + colors = listOf(Color(0xFF424BBD), Color(0xFF121212).copy(alpha = 0.04f)), )app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordGraphItem.kt (1)
101-127: Preview 함수명 일관성 고려컴포넌트명이
ProfileKeywordGraphItem으로 변경되었으나, Preview 함수명은ProfileKeywordGraphPreview로 유지되어 있습니다. 일관성을 위해ProfileKeywordGraphItemPreview로 변경을 고려해 주세요.💡 이름 변경 제안
`@Preview`(showBackground = false) `@Composable` -private fun ProfileKeywordGraphPreview() { +private fun ProfileKeywordGraphItemPreview() {app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordSection.kt (1)
79-83: 접근성: 아이콘에contentDescription추가 권장새로고침 아이콘의
contentDescription이null로 설정되어 있습니다. 스크린 리더 사용자를 위해 의미 있는 설명을 추가하는 것이 좋습니다.♻️ 제안하는 수정
Icon( imageVector = ImageVector.vectorResource(R.drawable.ic_refresh), - contentDescription = null, + contentDescription = "취향 키워드 업데이트", tint = FlintTheme.colors.secondary400, )app/src/main/java/com/flint/core/designsystem/component/listView/CollectionSection.kt (2)
32-40: Compose 컨벤션:modifier파라미터 위치Compose 컨벤션에서는
modifier가 필수 파라미터 다음, 선택적 파라미터 앞에 위치하는 것이 권장됩니다. 현재isAllVisible,onAllClick등 필수 콜백들 사이에 위치해 있습니다.
93-100:itemsIndexed대신items와key파라미터 사용 권장
index가 사용되지 않으므로items()로 충분하며,key를 지정하면 리스트 변경 시 불필요한 recomposition을 방지할 수 있습니다.CollectionModel에는 고유 식별자인collectionId가 있으므로 이를 key로 활용할 수 있습니다.♻️ 제안하는 수정
-itemsIndexed(collectionModelList) { _, item -> +items( + items = collectionModelList, + key = { it.collectionId } +) { item -> CollectionItem( collectionModel = item, onItemClick = { id -> onItemClick(id) }, ) }app/src/main/java/com/flint/presentation/profile/ProfileScreen.kt (1)
76-83:onRefreshClick이 빈 람다로 설정됨
ProfileKeywordSection의onRefreshClick이{}로 전달되어 새로고침 버튼이 아무 동작도 하지 않습니다. 향후 구현 예정이라면 TODO 주석을 추가하는 것이 좋습니다.♻️ TODO 주석 추가 제안
ProfileKeywordSection( nickname = userName, keywordList = keywordList, - onRefreshClick = {}, + onRefreshClick = {}, // TODO: 키워드 새로고침 기능 구현 modifier = Modifier.fillMaxWidth(), )app/src/main/java/com/flint/core/designsystem/component/listView/SavedContentsSection.kt (3)
94-101:itemsIndexed대신items와key사용 권장
CollectionSection과 동일하게,index가 사용되지 않으므로items()와key를 사용하는 것이 좋습니다.♻️ 제안하는 수정
-itemsIndexed(contentModelList) { _, item -> +items( + items = contentModelList, + key = { it.contentId } +) { item -> SavedContentItem( contentModel = item, onItemClick = { contentId -> onItemClick(contentId) }, ) }
106-121: Preview 함수명 불일치함수명이
PreviewHomeSavedContents로 "Home" 접두사가 포함되어 있지만, 컴포넌트명은SavedContentsSection입니다. 일관성을 위해SavedContentsSectionPreview로 변경하는 것이 좋습니다.♻️ 제안하는 수정
`@Preview` `@Composable` -private fun PreviewHomeSavedContents() { +private fun SavedContentsSectionPreview() {
31-104:CollectionSection과 코드 중복 가능성
SavedContentsSection과CollectionSection의 헤더 Row, 아이콘 처리, LazyRow 패턴이 거의 동일합니다. 향후 유지보수성을 위해 공통 헤더 컴포넌트를 추출하는 것을 고려해 볼 수 있습니다.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (13)
app/src/main/java/com/flint/core/designsystem/component/listView/CollectionSection.ktapp/src/main/java/com/flint/core/designsystem/component/listView/SavedContentsSection.ktapp/src/main/java/com/flint/domain/model/CollectionModel.ktapp/src/main/java/com/flint/domain/model/ContentModel.ktapp/src/main/java/com/flint/domain/model/PreferenceKeywordModel.ktapp/src/main/java/com/flint/presentation/home/HomeScreen.ktapp/src/main/java/com/flint/presentation/home/component/HomeRecommendCollection.ktapp/src/main/java/com/flint/presentation/home/component/HomeSavedContents.ktapp/src/main/java/com/flint/presentation/profile/ProfileScreen.ktapp/src/main/java/com/flint/presentation/profile/component/ProfileKeywordGraphItem.ktapp/src/main/java/com/flint/presentation/profile/component/ProfileKeywordSection.ktapp/src/main/java/com/flint/presentation/profile/component/ProfileTopSection.ktapp/src/main/res/drawable/ic_refresh.xml
💤 Files with no reviewable changes (2)
- app/src/main/java/com/flint/presentation/home/component/HomeRecommendCollection.kt
- app/src/main/java/com/flint/presentation/home/component/HomeSavedContents.kt
🧰 Additional context used
🧠 Learnings (3)
📚 Learning: 2026-01-13T17:03:16.816Z
Learnt from: giovannijunseokim
Repo: imflint/Flint-Android PR: 72
File: app/src/main/java/com/flint/core/designsystem/component/collection/PeopleBottomSheet.kt:85-98
Timestamp: 2026-01-13T17:03:16.816Z
Learning: In PeopleBottomSheet.kt, the `people` list passed to LazyColumn is static and does not change while the bottom sheet is visible, so a key parameter is not required for the items() call.
Applied to files:
app/src/main/java/com/flint/presentation/home/HomeScreen.ktapp/src/main/java/com/flint/presentation/profile/component/ProfileKeywordGraphItem.ktapp/src/main/java/com/flint/presentation/profile/ProfileScreen.kt
📚 Learning: 2026-01-13T19:02:41.580Z
Learnt from: chanmi1125
Repo: imflint/Flint-Android PR: 77
File: app/src/main/java/com/flint/presentation/collectioncreate/component/CollectionCreateThumbnail.kt:45-72
Timestamp: 2026-01-13T19:02:41.580Z
Learning: In `app/src/main/java/com/flint/presentation/collectioncreate/component/CollectionCreateThumbnail.kt`, the height difference between CollectionCreateEmptyThumbnail (no aspectRatio) and CollectionCreateFillThumbnail (aspectRatio 1.5f / 1f) is intentional design - the empty state should not have the same aspectRatio as the filled state.
Applied to files:
app/src/main/java/com/flint/core/designsystem/component/listView/CollectionSection.kt
📚 Learning: 2026-01-13T19:02:56.195Z
Learnt from: chanmi1125
Repo: imflint/Flint-Android PR: 77
File: app/src/main/java/com/flint/presentation/collectioncreate/component/CollectionCreateThumbnail.kt:45-72
Timestamp: 2026-01-13T19:02:56.195Z
Learning: In `app/src/main/java/com/flint/presentation/collectioncreate/component/CollectionCreateThumbnail.kt`, the aspect ratio difference between `CollectionCreateEmptyThumbnail` (no aspect ratio) and `CollectionCreateFillThumbnail` (1.5f / 1f) is intentional by design.
Applied to files:
app/src/main/java/com/flint/core/designsystem/component/listView/CollectionSection.kt
🧬 Code graph analysis (5)
app/src/main/java/com/flint/presentation/profile/component/ProfileTopSection.kt (2)
app/src/main/java/com/flint/core/designsystem/component/image/ProfileImage.kt (1)
ProfileImage(11-29)app/src/main/java/com/flint/core/designsystem/theme/Theme.kt (1)
FlintTheme(8-16)
app/src/main/java/com/flint/core/designsystem/component/listView/CollectionSection.kt (1)
app/src/main/java/com/flint/core/designsystem/theme/Theme.kt (1)
FlintTheme(8-16)
app/src/main/java/com/flint/presentation/profile/ProfileScreen.kt (5)
app/src/main/java/com/flint/presentation/profile/component/ProfileTopSection.kt (1)
ProfileTopSection(31-97)app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordSection.kt (1)
ProfileKeywordSection(33-102)app/src/main/java/com/flint/core/designsystem/component/listView/CollectionSection.kt (1)
CollectionSection(31-103)app/src/main/java/com/flint/core/designsystem/component/listView/SavedContentsSection.kt (1)
SavedContentsSection(31-104)app/src/main/java/com/flint/core/designsystem/theme/Theme.kt (1)
FlintTheme(8-16)
app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordSection.kt (4)
app/src/main/java/com/flint/domain/model/PreferenceKeywordModel.kt (1)
rotateKeywordByRank(17-32)app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordChip.kt (1)
ProfileKeywordChip(26-47)app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordGraphItem.kt (1)
ProfileKeywordGraphItem(29-75)app/src/main/java/com/flint/core/designsystem/theme/Theme.kt (1)
FlintTheme(8-16)
app/src/main/java/com/flint/core/designsystem/component/listView/SavedContentsSection.kt (2)
app/src/main/java/com/flint/core/designsystem/component/listItem/SavedContentItem.kt (1)
SavedContentItem(24-74)app/src/main/java/com/flint/core/designsystem/theme/Theme.kt (1)
FlintTheme(8-16)
⏰ 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). (2)
- GitHub Check: PR Lint Check
- GitHub Check: PR Build Check
🔇 Additional comments (9)
app/src/main/res/drawable/ic_refresh.xml (1)
1-13: LGTM!새로 고침 아이콘이 올바르게 정의되었습니다. 벡터 드로어블 설정이 적절합니다.
app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordGraphItem.kt (1)
29-75: LGTM!파라미터명 변경(
percent→percentage)과 텍스트 정렬 개선이 잘 적용되었습니다.TextAlign.End와widthIn(min = 32.dp)조합으로 퍼센트 값이 일관되게 정렬됩니다.app/src/main/java/com/flint/domain/model/PreferenceKeywordModel.kt (1)
34-91: FakeList 정의가 적절합니다.샘플 데이터가 명확하게 정의되어 있으며,
FakeList2가FakeList1을copy()로 재사용하는 패턴은 중복을 줄이는 좋은 접근입니다.app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordSection.kt (1)
33-44: LGTM!
remember(keywordList)를 사용하여rotateKeywordByRank결과를 메모이제이션하는 것은 불필요한 재계산을 방지하는 좋은 패턴입니다.app/src/main/java/com/flint/presentation/profile/ProfileScreen.kt (2)
35-41:ProfileRoute에서onFilmItemClick콜백 미전달
ProfileScreen의onFilmItemClick파라미터가ProfileRoute에서 전달되지 않아 기본값 빈 람다{}가 사용됩니다. Line 52의 TODO 주석("바텀시트 띄우기")과 관련이 있는 것으로 보이지만, 향후 구현 시 누락되지 않도록 주의가 필요합니다.
57-64: LGTM!
LazyColumn을 사용한 섹션 기반 레이아웃 구조가 잘 설계되어 있습니다.overscrollEffect = null로 오버스크롤 효과를 비활성화하고contentPadding으로 하단 여백을 확보한 것이 적절합니다.app/src/main/java/com/flint/presentation/home/HomeScreen.kt (3)
18-28: LGTM!
ImmutableList로의 마이그레이션은 Compose의 안정성(stability) 측면에서 좋은 선택입니다. 불필요한 리컴포지션을 방지할 수 있습니다.
48-61: LGTM!파라미터가
ImmutableList로 잘 마이그레이션되었고, 기본값으로persistentListOf()를 사용한 것이 적절합니다.
88-129: LGTM!새로운
CollectionSection과SavedContentsSection컴포넌트가 적절하게 사용되었습니다.isAllVisible플래그에 따른 조건부 표시 로직도 명확합니다.
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
| fun rotateKeywordByRank(keywordList: ImmutableList<PreferenceKeywordModel>): ImmutableList<PreferenceKeywordModel> { | ||
| if (keywordList.size < 2) return keywordList | ||
|
|
||
| val sortedByRank = keywordList.sortedBy { it.rank } | ||
|
|
||
| // rank 1, 2, 3 (상위 3개) → Large | ||
| val topRanks = sortedByRank.take(3) | ||
| // rank 4, 5, 6 (하위 3개) → Small | ||
| val bottomRanks = sortedByRank.drop(3) | ||
|
|
||
| // 번갈아 배치: 1, 4, 2, 5, 3, 6 | ||
| return topRanks | ||
| .zip(bottomRanks) | ||
| .flatMap { (top, bottom) -> listOf(top, bottom) } | ||
| .toPersistentList() | ||
| } |
There was a problem hiding this comment.
rotateKeywordByRank에서 리스트 크기가 6 미만일 때 요소 누락 가능성
zip은 두 리스트 중 짧은 쪽의 길이만큼만 쌍을 생성합니다. 따라서 keywordList의 크기가 4 또는 5일 경우, topRanks의 일부 요소가 결과에서 누락됩니다.
예시: 5개 아이템 → topRanks=[1,2,3], bottomRanks=[4,5] → 결과: [1,4,2,5] (3번 누락)
🐛 제안하는 수정 방안
fun rotateKeywordByRank(keywordList: ImmutableList<PreferenceKeywordModel>): ImmutableList<PreferenceKeywordModel> {
if (keywordList.size < 2) return keywordList
val sortedByRank = keywordList.sortedBy { it.rank }
// rank 1, 2, 3 (상위 3개) → Large
val topRanks = sortedByRank.take(3)
// rank 4, 5, 6 (하위 3개) → Small
val bottomRanks = sortedByRank.drop(3)
// 번갈아 배치: 1, 4, 2, 5, 3, 6
- return topRanks
- .zip(bottomRanks)
- .flatMap { (top, bottom) -> listOf(top, bottom) }
- .toPersistentList()
+ val result = mutableListOf<PreferenceKeywordModel>()
+ val maxSize = maxOf(topRanks.size, bottomRanks.size)
+ for (i in 0 until maxSize) {
+ if (i < topRanks.size) result.add(topRanks[i])
+ if (i < bottomRanks.size) result.add(bottomRanks[i])
+ }
+ return result.toPersistentList()
}🤖 Prompt for AI Agents
In `@app/src/main/java/com/flint/domain/model/PreferenceKeywordModel.kt` around
lines 17 - 32, rotateKeywordByRank currently zips topRanks and bottomRanks which
drops any unmatched elements when sizes differ (e.g., 4 or 5 items), so change
the assembly logic to interleave pairs from topRanks and bottomRanks and then
append any remaining elements from the longer list; specifically, in
rotateKeywordByRank after computing topRanks and bottomRanks use the zip result
plus the remainder of topRanks or bottomRanks (starting at the zip size) to
ensure no element is lost, preserving original rank order within each subgroup.
There was a problem hiding this comment.
정책상 서버에서 무조건 키워드 6개를 넘겨주는데, 그래도 처리해야할까?
There was a problem hiding this comment.
@nahy-512, 서버에서 항상 6개를 보장한다면 현재 구현으로도 정상 동작합니다!
다만 몇 가지 옵션을 고려해보실 수 있습니다:
- 현재대로 유지 - 서버 정책이 명확하다면 추가 처리 없이 그대로 사용
- 주석 추가 - 함수 상단에 "서버에서 항상 6개 키워드 보장" 명시
- 방어적 처리 - 혹시 모를 변경이나 테스트 시나리오 대비
예시 (주석 추가):
/**
* 키워드를 rank 기준으로 번갈아 배치 (1,4,2,5,3,6)
* `@param` keywordList 서버에서 보장하는 6개의 키워드 리스트
*/
fun rotateKeywordByRank(keywordList: ImmutableList<PreferenceKeywordModel>): ImmutableList<PreferenceKeywordModel> {또는 assertion 추가:
fun rotateKeywordByRank(keywordList: ImmutableList<PreferenceKeywordModel>): ImmutableList<PreferenceKeywordModel> {
require(keywordList.size == 6) { "서버는 항상 6개의 키워드를 제공해야 합니다" }
// ... 기존 로직팀의 코딩 스타일에 맞게 선택하시면 됩니다. 현재대로 진행하셔도 문제없습니다! 👍
| KeywordGraphLayout( | ||
| keywordList = keywordList.subList(0, MAX), | ||
| modifier = Modifier.fillMaxWidth(), | ||
| ) |
There was a problem hiding this comment.
subList 호출 시 IndexOutOfBoundsException 발생 가능
keywordList의 크기가 MAX(3) 미만일 경우 subList(0, MAX)에서 예외가 발생합니다.
🐛 제안하는 수정 방안
KeywordGraphLayout(
- keywordList = keywordList.subList(0, MAX),
+ keywordList = keywordList.take(MAX).toImmutableList(),
modifier = Modifier.fillMaxWidth(),
)take()를 사용하면 리스트 크기가 MAX보다 작아도 안전하게 처리됩니다. toImmutableList()는 kotlinx.collections.immutable에서 import해야 합니다.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| KeywordGraphLayout( | |
| keywordList = keywordList.subList(0, MAX), | |
| modifier = Modifier.fillMaxWidth(), | |
| ) | |
| KeywordGraphLayout( | |
| keywordList = keywordList.take(MAX).toImmutableList(), | |
| modifier = Modifier.fillMaxWidth(), | |
| ) |
🤖 Prompt for AI Agents
In
`@app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordSection.kt`
around lines 97 - 100, Calling keywordList.subList(0, MAX) can throw
IndexOutOfBoundsException when keywordList.size < MAX; update the call in
ProfileKeywordSection (where KeywordGraphLayout is invoked) to pass
keywordList.take(MAX) (and convert to immutable with toImmutableList() if an
immutable list is required), ensuring to import toImmutableList() from
kotlinx.collections.immutable when used.
| keyword: String, | ||
| preferenceType: PreferenceType, | ||
| percent: Int, | ||
| percentage: Int, |
There was a problem hiding this comment.
p3: 사용될 때 Int 그 자체로써 쓰이는 경우는 없는 것 같아요. Float으로 받아도 좋을 것 같습니다. 👍
| isFliner: Boolean, | ||
| modifier: Modifier = Modifier, | ||
| ) { | ||
| Column { |
There was a problem hiding this comment.
p3: 최상위를 Column으로 감싸신 이유가 있으실까요? 👀
| val gradient = | ||
| Brush.verticalGradient( | ||
| colors = listOf(Color(0xFF424BBD).copy(1f), Color(0xFF121212).copy(alpha = 0.04f)), | ||
| ) |
There was a problem hiding this comment.
p3: 이 부분도 FlintTheme.colors에 추가하는건 어떨까요? 🤔
There was a problem hiding this comment.
디자인 시스템에 정의되지 않은 색상이라 '해당 화면에만 쓰이는 건가?' 싶어서 컴포넌트 단에서 정의를 했었는데요.
리뷰어에 내용 적는 걸 까먹었네요ㅠㅠ 디자인 쌤들께 여쭤보고 추가 부탁드렸습니다!
수정해두겠습니다!
| @Preview(showBackground = false) | ||
| @Composable | ||
| private fun ProfileTopSectionPreview() { | ||
| FlintTheme { | ||
| ProfileTopSection( | ||
| userName = "안두콩", | ||
| profileUrl = "", | ||
| isFliner = false, | ||
| ) | ||
| } | ||
| } |
There was a problem hiding this comment.
p3: 플리너일 경우의 Preview도 볼 수 있으면 좋을 것 같습니다. 👍
| @Preview(showBackground = false) | |
| @Composable | |
| private fun ProfileTopSectionPreview() { | |
| FlintTheme { | |
| ProfileTopSection( | |
| userName = "안두콩", | |
| profileUrl = "", | |
| isFliner = false, | |
| ) | |
| } | |
| } | |
| @Preview(showBackground = false) | |
| @Composable | |
| private fun ProfileTopSectionPreview1() { | |
| FlintTheme { | |
| ProfileTopSection( | |
| userName = "안두콩", | |
| profileUrl = "", | |
| isFliner = false, | |
| ) | |
| } | |
| } | |
| @Preview(showBackground = false) | |
| @Composable | |
| private fun ProfileTopSectionPreview2() { | |
| FlintTheme { | |
| ProfileTopSection( | |
| userName = "안두콩", | |
| profileUrl = "", | |
| isFliner = true, | |
| ) | |
| } | |
| } | |
There was a problem hiding this comment.
감사합니다ㅎㅎ 케이스 추가해보겠습니다!
| import com.flint.domain.type.KeywordType | ||
| import kotlinx.collections.immutable.ImmutableList | ||
|
|
||
| private const val MAX = 3 |
There was a problem hiding this comment.
p3: 어떤 값의 최댓값인지 써주시면 좋을 것 같아요. 👍
ex) KEYWORD_LIST_MAX_SIZE
There was a problem hiding this comment.
나중에 수정해야지 해놓고 까먹었네요ㅎㅎ 앞으로 PR 올린 뒤에 코드 확인을 한번 더 해봐야겠어요ㅎㅎ
| modifier = | ||
| Modifier | ||
| .align(Alignment.Center), |
There was a problem hiding this comment.
p3: 이미 상위 레이아웃 Box에서 Alignment.Center를 정해준 것 같아요. 👀
| onAllClick() | ||
| }, | ||
| ) | ||
| if (isAllVisible) { |
There was a problem hiding this comment.
p3: 이 분기처리가 도입된 배경이 있나요? 화면설계서나 피그마에선 못 찾았어요. 😢
There was a problem hiding this comment.
홈에서 추천 컬렉션, 최근 저장한 콘텐츠는 상세보기로 못 가게 화살표가 아예 없어서용!!
home/data stacked user 부분에서 확인해보실 수 있습니당
| Icon( | ||
| imageVector = ImageVector.vectorResource(R.drawable.ic_more), | ||
| contentDescription = null, | ||
| tint = Color.Unspecified, |
There was a problem hiding this comment.
p3: 코드만 보는 입장에서(ex: 깃허브 환경) 어떤 색인지 알기 어려울 것 같아요. FlintTheme.colors.white와 같은 식으로 명시해주는건 어떨까요?
There was a problem hiding this comment.
오앙 IDE에서 바로 흰색임을 확인할 수 있어서 별도로 설정하지 않았는데, 원본 아이콘 색상이 뭔지 코드에서 바로 알긴 힘드니 확실히 한번 더 명시해주는 편이 좋겠네요. 감사합니다!
| horizontalArrangement = Arrangement.spacedBy(12.dp), | ||
| modifier = Modifier.fillMaxWidth(), | ||
| ) { | ||
| itemsIndexed(contentModelList) { _, item -> |
There was a problem hiding this comment.
p3: index를 사용하지 않고 있는 것 같아요. itemsIndexed 대신 items를 활용할 수 있지 않을까요? 👀
| modifier | ||
| .fillMaxWidth(), | ||
| ) { | ||
| FlowRow( |
There was a problem hiding this comment.
p3: FlowRow처럼 자연스럽게 넘어가야 하는 상황이 아니라면 단순 Row를 쓰는게 더 편할 것 같아요. 👍
There was a problem hiding this comment.
안그래도 2~3줄 처리에 컴포넌트 배치가 달라져서 조금 고민이었는데, 글자수 기반으로 단순 Row로 구현하는 방식이 더 나을 수도 있겠네요. 감사합니다!
2 participants
📮 관련 이슈
📌 작업 내용
📸 스크린샷
2026-01-16.1.04.03.mov
😅 미구현
🫛 To. 리뷰어
FlowRow로 구현했는데 2줄일 때랑 3줄일 때랑 컴포넌트 배치가 아예 달라서 고민이네요.
Summary by CodeRabbit
릴리스 노트
New Features
Refactor
✏️ Tip: You can customize this high-level summary in your review settings.