[Feat] 1차 QA 수정사항 반영

[ckals413]

📮 관련 이슈

• closed #이슈번호

📌 작업 내용

• 온보딩 콘텐츠 검색 화면 로딩 인디케이터 추가 (초기 진입 및 장르 칩 선택 시)
• 컬렉션 리스트 아이템 제목/설명 한글 줄바꿈 개선 (zero-width space 삽입)
• 온보딩 콘텐츠 아이템 제목 한글 줄바꿈 개선
• 취향 키워드 업데이트 기준 텍스트 수정 (10개 → 20개)

😅 미구현ㅂ- [ ] 온보딩 done 회원 정보 저장

🫛 To. 리뷰어

• 온보딩 done 회원 정보 저장은 추가 논의가 필요해 남겨뒀습니다

Summary by CodeRabbit

• New Features

  • 한글 제목/설명과 영화 제목에서 줄바꿈이 더 자연스럽게 보이도록 텍스트 표시가 개선되었습니다.
  • 온보딩 콘텐츠 목록의 로딩 상태에 전체 폭 로딩 인디케이터가 표시됩니다.

• Bug Fixes

  • 긴 한글 문구가 화면에서 더 잘 끊겨 보이도록 줄바꿈 처리가 강화되었습니다.
  • 프로필 안내 문구의 업데이트 가능 조건이 최신 기준으로 반영되었습니다.

[coderabbitai]

📝 Walkthrough

Walkthrough

CollectionFileItem과 OnboardingContentItem에 한글 음절 뒤에 zero-width space를 삽입하는 addKoreanLineBreaks() 확장 함수를 각각 추가하여 title/description 텍스트에 적용합니다. OnboardingContentScreen의 로딩 상태에 FlintLoadingIndicator를 추가하고, ProfileKeywordSection의 업데이트 허용 기준 작품 수를 20개로 변경합니다.

Changes

한글 줄바꿈, 로딩 UI 및 문구 수정

Layer / File(s)	Summary
addKoreanLineBreaks() 구현 및 적용 app/.../collectionlist/component/CollectionFileItem.kt, app/.../onboarding/component/OnboardingContentItem.kt	각 파일에 private fun String.addKoreanLineBreaks()를 추가하여 한글 음절(0xAC00..0xD7A3) 뒤에 U+200B를 삽입하고, title/description Text에 변환된 문자열을 전달합니다.
온보딩 로딩 인디케이터 추가 app/.../onboarding/OnboardingContentScreen.kt	UiState.Loading 분기에서 그리드 3칸을 span하는 컨테이너 안에 FlintLoadingIndicator를 렌더링하도록 변경합니다.
프로필 키워드 기준 문구 수정 app/.../profile/component/ProfileKeywordSection.kt	InfoModalTrigger의 텍스트에서 저장 작품 업데이트 허용 기준을 "10개 이상"에서 "20개 이상"으로 변경합니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• imflint/Flint-Android#111: CollectionFileItem.kt에서 텍스트 레이아웃 관련 수정을 함께 다루는 PR으로, 이번 PR의 addKoreanLineBreaks() 적용과 동일한 컴포넌트를 변경합니다.

Suggested labels

📱 UI, Style 🎨

Suggested reviewers

• nahy-512
• kimjw2003
• giovannijunseokim

Poem

🐰 토끼가 글자를 톡톡 두드려요,
한글 음절 사이에 숨결을 불어넣죠.
제로폭 요정이 줄바꿈 길 열어주고,
로딩 빙글빙글 이제 예쁘게 돌아가요.
스무 개의 작품, 새 기준이 되었네요! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 11.11% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR의 핵심인 QA 수정사항 반영을 잘 나타내며, 변경 내용과 직접 관련된 간결한 제목입니다.
Description check	✅ Passed	필수 섹션을 대부분 포함하고 있으며 작업 내용과 미구현, 리뷰어 메모가 있어 템플릿 요구사항을 대체로 충족합니다.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch FLT-25-1차-qa

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

🧹 Nitpick comments (1)

app/src/main/java/com/flint/presentation/onboarding/component/OnboardingContentItem.kt (1)

120-125: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

중복된 줄바꿈 헬퍼는 공용 유틸로 합쳐주세요.

같은 addKoreanLineBreaks() 구현이 CollectionFileItem.kt에도 별도로 들어가 있어서, 이후 문자 범위나 zero-width space 처리 방식을 바꿀 때 두 파일이 쉽게 어긋납니다. presentation 공용 확장 함수로 올리고 두 컴포넌트가 같이 쓰도록 정리하는 편이 안전합니다.

🤖 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/java/com/flint/presentation/onboarding/component/OnboardingContentItem.kt`
around lines 120 - 125, The addKoreanLineBreaks() logic is duplicated in
OnboardingContentItem and CollectionFileItem, so move this String extension into
a shared presentation-level utility/extension file and have both components use
the same implementation. Keep the symbol name addKoreanLineBreaks() recognizable
in the new shared location, then remove the local copies from each component so
the Hangul range and zero-width space behavior stay consistent.

🤖 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/java/com/flint/presentation/onboarding/component/OnboardingContentItem.kt`:
- Around line 120-125: The addKoreanLineBreaks() logic is duplicated in
OnboardingContentItem and CollectionFileItem, so move this String extension into
a shared presentation-level utility/extension file and have both components use
the same implementation. Keep the symbol name addKoreanLineBreaks() recognizable
in the new shared location, then remove the local copies from each component so
the Hangul range and zero-width space behavior stay consistent.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 21cb3dfc-1225-470f-8a73-8597821f6d83

📥 Commits

Reviewing files that changed from the base of the PR and between 84a3f0a and d4ef4f9.

📒 Files selected for processing (4)

• app/src/main/java/com/flint/presentation/collectionlist/component/CollectionFileItem.kt
• app/src/main/java/com/flint/presentation/onboarding/OnboardingContentScreen.kt
• app/src/main/java/com/flint/presentation/onboarding/component/OnboardingContentItem.kt
• app/src/main/java/com/flint/presentation/profile/component/ProfileKeywordSection.kt