Conversation
- 인기 피드 내용(`tv_popular_feed_content`)의 상단 여백을 4dp에서 8dp로 변경
- `item_popular_feed_slot.xml`의 패딩 값을 미세 조정하고 텍스트 뷰 너비를 `0dp`(match constraint)로 변경하여 레이아웃 최적화 - `PopularFeedsViewHolder`의 `ellipsizeByLength` 메서드 로직을 공백을 제외한 글자 수 기준으로 계산하도록 수정 - 제목 생략 시 노출되는 말줄임표 기호를 `...`에서 `…`으로 변경
- 인기 피드 노출 개수를 최대 6개로 제한하기 위해 `HOME_POPULAR_FEED_MAX_COUNT` 상수 추가 - `popularFeeds` 데이터를 페이지 단위로 가공하는 `toHomePopularFeedPages` 확장 함수 구현 - `HomeViewModel` 내 데이터 로딩 및 업데이트 로직에 개수 제한 및 청크 로직 통합 적용
Walkthrough
Changes홈 화면 인기 피드 QA 수정
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 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📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
- 불필요한 배경 리소스(`bg_popular_novel_translucent_radius_14dp`) 삭제 및 `item_popular_novel`의 배경을 공통 리소스로 교체 - 인기 작품 아이템 하단 영역 배경(`bg_popular_novel_bottom_radius_14dp`)에 `layer-list`를 적용하여 색상 및 투명도 레이어 수정
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
app/src/main/res/drawable/bg_popular_novel_bottom_radius_14dp.xml (1)
4-4: ⚡ Quick win하드코딩된 색상 값을
@color리소스로 추출하는 것을 고려하세요.현재
#F6F6F8과#B2FFFFFF색상이 하드코딩되어 있습니다. 디자인 시스템의 일관성과 유지보수성을 위해 이 색상들을colors.xml에 정의하고 참조하는 것이 좋습니다.♻️ 색상 리소스 추출 예시
colors.xml에 색상 추가:<color name="popular_novel_bottom_base">`#F6F6F8`</color> <color name="popular_novel_bottom_overlay">`#B2FFFFFF`</color>이 파일에 적용:
<item> <shape android:shape="rectangle"> - <solid android:color="`#F6F6F8`" /> + <solid android:color="`@color/popular_novel_bottom_base`" /> <corners android:bottomLeftRadius="14dp" android:bottomRightRadius="14dp" /> </shape> </item> <item> <shape android:shape="rectangle"> - <solid android:color="`#B2FFFFFF`" /> + <solid android:color="`@color/popular_novel_bottom_overlay`" /> <corners android:bottomLeftRadius="14dp" android:bottomRightRadius="14dp" /> </shape> </item>Also applies to: 13-13
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@app/src/main/res/drawable/bg_popular_novel_bottom_radius_14dp.xml` at line 4, Extract the hardcoded color values from the drawable file into Android color resources. In app/src/main/res/values/colors.xml (or create it if it doesn't exist), add color definitions for popular_novel_bottom_base with value `#F6F6F8` and popular_novel_bottom_overlay with value `#B2FFFFFF`. Then in app/src/main/res/drawable/bg_popular_novel_bottom_radius_14dp.xml at line 4, replace the hardcoded solid color android:color="`#F6F6F8`" with android:color="`@color/popular_novel_bottom_base`" and at line 13, replace the hardcoded color android:color="`#B2FFFFFF`" with android:color="`@color/popular_novel_bottom_overlay`".
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@app/src/main/res/drawable/bg_popular_novel_bottom_radius_14dp.xml`:
- Line 4: Extract the hardcoded color values from the drawable file into Android
color resources. In app/src/main/res/values/colors.xml (or create it if it
doesn't exist), add color definitions for popular_novel_bottom_base with value
`#F6F6F8` and popular_novel_bottom_overlay with value `#B2FFFFFF`. Then in
app/src/main/res/drawable/bg_popular_novel_bottom_radius_14dp.xml at line 4,
replace the hardcoded solid color android:color="`#F6F6F8`" with
android:color="`@color/popular_novel_bottom_base`" and at line 13, replace the
hardcoded color android:color="`#B2FFFFFF`" with
android:color="`@color/popular_novel_bottom_overlay`".
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: fc9eb257-69a3-44c7-875c-b3ae60f1e489
📒 Files selected for processing (3)
app/src/main/res/drawable/bg_popular_novel_bottom_radius_14dp.xmlapp/src/main/res/drawable/bg_popular_novel_translucent_radius_14dp.xmlapp/src/main/res/layout/item_popular_novel.xml
💤 Files with no reviewable changes (1)
- app/src/main/res/drawable/bg_popular_novel_translucent_radius_14dp.xml
✅ Files skipped from review due to trivial changes (1)
- app/src/main/res/layout/item_popular_novel.xml
- `PopularNovelTextExtensions.kt`: 제목 생략 처리 시 공백을 제외한 화면에 보이는 글자 수를 기준으로 하도록 `takeWithEllipsis` 로직 고도화 - `strings.xml`: 제목 생략 기호를 일반 마침표 3개(...)에서 생략 부호(…)로 변경
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@app/src/main/java/com/into/websoso/ui/main/home/adpater/PopularNovelTextExtensions.kt`:
- Around line 50-55: The truncation logic in the takeWhile lambda correctly
implements the character counting mechanism using maxLength and visibleLength,
but the actual maxLength value being passed is 17 when the requirement specifies
a limit of 16 characters (excluding whitespace). Locate where maxLength is
defined or passed as a parameter to this function and change the value from 17
to 16 to align with the PR requirement of displaying only 16 characters
excluding whitespace.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 7e7e8c0c-1a7a-4d33-9a28-99bb98d1283b
📒 Files selected for processing (2)
app/src/main/java/com/into/websoso/ui/main/home/adpater/PopularNovelTextExtensions.ktcore/resource/src/main/res/values/strings.xml
- `PopularNovelEntity.toAuthorStatus`에서 작가명이 최대 길이를 초과할 경우 말줄임표(...)를 표시하도록 로직 수정
- `item_popular_novel.xml`: 소설 커버 이미지와 장르 아이콘을 `MaterialCardView`로 감싸 그림자(`cardElevation`) 및 라운드 처리 적용 - 커버 이미지의 라운드 처리 방식을 커스텀 속성에서 `MaterialCardView` 속성(`cardCornerRadius`)으로 변경 - 카드 뷰 내부 아이콘들의 배치 방식을 `ConstraintLayout` 제약 조건에서 `layout_gravity` 활용 방식으로 수정
m6z1
approved these changes
Jun 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌𝘐𝘴𝘴𝘶𝘦𝘴
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
<지금 뜨는글> 소설 제목과 본문 사이의 간격에 4dp -> 8dp 로 수정됨에 따라 8dp로 수정
<지금 뜨는글> 제목 글자 수 최대 16자, 그 이후 텍스트 자르기 : 공백 제외 후 보이는 제목 글자수 16자만 보이게끔 수정
<지금 뜨는 글> 현재 서버에서 받아오는 글이 9개이나 6개만 보이게 하도록 수정
<오늘의 발견> 하단 한마디 부분에 블러 적용된 FFFFF 이미지로 수정 및 투명도는 70% 적용 ->
피그마의 색상 #FFFFFFB2가 반영되어있었으나 노랗게 보이는 문제가 있어 의도한 색상이 보이도록 밑색을 넣은 뒤 투명도를 설정하여 변경
<오늘의 발견> 소설 제목 17자까지만 보이게 수정 : 공백 제외 보여지는 글자수가 17자가 되도록 수정
<오늘의 발견> 작품 표지 아래 썸네일 그림자 처리
📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵
지금 뜨는 글
오늘의 발견
Screen_recording_20260617_170207.mp4
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴
Summary by CodeRabbit
릴리스 노트
…형태로 더 자연스럽게 변경했습니다.