Skip to content

fix: 하차 글자 밀리는 문제 수정#851

Merged
devfeijoa merged 1 commit intodevelopfrom
fix/843
Mar 29, 2026
Merged

fix: 하차 글자 밀리는 문제 수정#851
devfeijoa merged 1 commit intodevelopfrom
fix/843

Conversation

@devfeijoa
Copy link
Copy Markdown
Contributor

@devfeijoa devfeijoa commented Mar 25, 2026

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • s25 플러스에서 내 서재의 읽기 상태인 "하차" 글자가 밀리는 문제 수정

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

Screenshot_20260325_172134 Screenshot_20260325_172259

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

softwrap = false
maxLine = 1
그리고 공식문서에서 권장하는대로 ...으로 글이 축약되게하는
overflow = TextOverflow.Ellipsis 옵션을 넣어 빠르게 문제가 생기면 사용자와 개발자가 알아낼 수 있도록 하였습니다.
추후에도 문제가 여전히 있으면, 디자인 시스템의 텍스트 부분을 dp -> sp로 바꾸는 방법도 고려해보도록 하겠습니다

참고 자료 : https://developer.android.com/develop/ui/compose/text/configure-layout?hl=ko
https://developer.android.com/reference/kotlin/androidx/compose/ui/text/TextLayoutInput?hl=en

Summary by CodeRabbit

버그 수정

  • 읽음 상태 레이블의 텍스트 오버플로우 처리 개선: 레이블이 길 경우 한 줄로 제한되며 말줄임표(...)로 자동 축약됩니다.

@devfeijoa devfeijoa requested review from Sadturtleman, m6z1 and s9hn March 25, 2026 08:14
@devfeijoa devfeijoa self-assigned this Mar 25, 2026
@devfeijoa devfeijoa added 🔨 [FIX] 버그를 수정합니다. [👸 공주 은영] labels Mar 25, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 25, 2026

워크스루

LibraryListItem.ktReadStatusBadge 컴포넌트에서 Text 위젯의 텍스트 렌더링 설정을 수정했습니다. 소프트 래핑을 비활성화하고, 최대 라인 수를 1줄로 제한하며, 텍스트 오버플로우 시 말줄임표 처리를 추가했습니다.

변경사항

코호트 / 파일 요약
ReadStatusBadge 텍스트 렌더링 설정
feature/library/src/main/java/com/into/websoso/feature/library/component/LibraryListItem.kt
readStatus.label 텍스트에 softWrap = false, maxLines = 1, overflow = TextOverflow.Ellipsis 속성을 추가하여 텍스트 오버플로우 문제를 해결했습니다.

예상 코드 리뷰 소요 시간

🎯 1 (Trivial) | ⏱️ ~5 minutes

검토 제안자

  • s9hn
  • m6z1
  • Sadturtleman

🐰 글자가 밀려나가는 문제, 이제 말줄임표로 깔끔하게

한 줄로 쏙, 갤럭시도 행복해져

하차 표시, 이제 완벽해! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% 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 제목이 변경의 핵심인 '하차' 글자 밀리는 문제 수정을 명확하게 설명하고 있습니다.
Linked Issues check ✅ Passed 코드 변경사항이 #843 이슈의 요구사항을 완벽하게 충족합니다. Galaxy S25 Plus의 '하차' 텍스트 밀림 문제를 텍스트 렌더링 속성 조정으로 해결했습니다.
Out of Scope Changes check ✅ Passed 모든 코드 변경사항이 #843 이슈의 범위 내에 있으며, ReadStatusBadge의 Text 렌더링 속성 수정만 포함되어 있습니다.
Description check ✅ Passed PR 설명이 필수 섹션을 모두 포함하고 있으며, 이슈 번호, 작업 내용, 스크린샷, 리뷰어 커멘트가 잘 작성되어 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 fix/843

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

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@Sadturtleman Sadturtleman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!

@devfeijoa devfeijoa merged commit 99a7116 into develop Mar 29, 2026
3 checks passed
@devfeijoa devfeijoa deleted the fix/843 branch March 29, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[👸 공주 은영] 🔨 [FIX] 버그를 수정합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: 하차 글자 밀리는 문제 수정

3 participants