fix: 작품 상세 읽기 상태 막대 그래프가 비율대로 표시되지 않음

[Sadturtleman]

📌𝘐𝘴𝘴𝘶𝘦𝘴

• closed fix: 작품 상세 읽기 상태 막대 그래프가 비율대로 표시되지 않음 #953
  https://discord.com/channels/1185753783020568648/1537808439965786192

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

디자인 QA로 접수된 막대 그래프 이슈를 수정했습니다.

1. 채움 높이를 세 컬럼 모두에 적용

막대는 바깥 트랙(CardView)과 안쪽 채움(View) 2겹 구조이고, 안쪽 채움 높이로 비율을 표현합니다. 그런데 updateGraphUi()가 when (maxCountReadStatus())로 최댓값 컬럼 하나만 높이를 적용해, 나머지 두 컬럼은 XML 기본 높이가 그대로 남아 트랙을 꽉 채운 상태였습니다.

높이 계산(formattedHeight = viewHeight * count / maxCount)과 formattedUnifiedReviewCount()는 이미 세 컬럼을 모두 계산하고 있어, 적용 지점만 넓히고 강조(선택) 처리만 최댓값에 남겼습니다.

2. 미선택 채움 색상 #DFDFE3 → #EEEEF2

디자인의 UI/gray.70은 #EEEEF2인데 colors.xml에는 해당 색이 없었습니다. Compose 토큰에는 Gray70New = #EEEEF2로 이미 존재합니다.

3. 보는 중 막대 기본 높이 80dp → 100dp

세 컬럼 중 보는 중만 기본 높이가 달랐습니다.

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

작품 황후가 되고 싶은 여자 (보는 중 1 / 봤어요 3 / 하차 0) 기준 실측입니다.

항목	값	기대 채움	수정 전	수정 후
보는 중	1	100px (1/3)	240px	100px
봤어요	3	300px	300px	300px

수정 후에는 밝은 트랙 안에 아래쪽 1/3만 진한 회색으로 채워져 시안과 일치합니다.

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

Summary by CodeRabbit

• 개선 사항

  • 소설 정보 화면의 독서 상태 그래프 높이를 일관되게 조정했습니다.
  • ‘보고 있어요’ 상태 막대가 다른 상태 막대와 동일한 높이로 표시됩니다.
  • 독서 상태에 따른 그래프 표시가 더욱 안정적으로 갱신됩니다.

• 스타일

  • 선택되지 않은 독서 상태 그래프의 배경 색상을 보다 자연스러운 밝은 회색으로 변경했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 908053f4-f4c9-4731-9d18-84a380afedc4

📥 Commits

Reviewing files that changed from the base of the PR and between 2b500e0 and 6194ad0.

📒 Files selected for processing (4)

• app/src/main/java/com/into/websoso/ui/novelInfo/NovelInfoFragment.kt
• app/src/main/res/color/bg_novel_info_read_status_graph_selector.xml
• app/src/main/res/layout/fragment_novel_info.xml
• app/src/main/res/values/colors.xml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

읽기 상태 그래프가 세 상태의 계산된 높이를 항상 적용하도록 변경되었습니다. WATCHING 그래프의 기본 높이를 조정하고, 미선택 그래프 색상 리소스를 변경했습니다.

Changes

읽기 상태 그래프

Layer / File(s)	Summary
그래프 높이 적용 app/src/main/java/com/into/websoso/ui/novelInfo/NovelInfoFragment.kt	WATCHING, WATCHED, QUIT 그래프의 높이를 공통으로 갱신합니다. 각 분기는 상태별 그래프와 카운트, 텍스트 선택만 처리합니다.
그래프 표시 리소스 app/src/main/res/layout/fragment_novel_info.xml, app/src/main/res/values/colors.xml, app/src/main/res/color/bg_novel_info_read_status_graph_selector.xml	WATCHING 그래프의 기본 높이를 80dp에서 100dp로 변경했습니다. 미선택 색상 gray_70_EEEEF2를 추가하고 적용했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 6194a

This PR corrects the proportional sizing and styling of the novel reading-status bars without changing data, interfaces, or runtime behavior; no actionable merge-blocking risk remains beyond normal checks and review.

Poem

토끼가 막대 그래프를 살펴요
세 기둥 높이가 함께 움직여요
보는 중 막대도 가지런해요
옅은 색은 새 옷을 입어요
당근처럼 정확한 비율이에요

🚥 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%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (3 skipped: 3 …	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	[953]의 핵심 요구사항을 충족합니다. 세 컬럼에 비율 기반 높이를 적용하고, 최댓값 컬럼의 선택 강조를 유지했습니다. 미선택 색상을 #EEEEF2로 변경했고, 보는 중 막대의 기본 높이를 100dp로 조정했습니다. 최소 높이 규칙은 이슈 요구사항에 따라 순수 비율로 구현했습니다.
Out of Scope Changes check	✅ Passed	모든 변경이 [953]의 읽기 상태 그래프 수정 범위에 포함됩니다. 그래프 높이 계산 적용, 색상 리소스 변경, 보는 중 막대 높이 조정만 수행했으며 관련 없는 변경은 확인되지 않습니다.
Title check	✅ Passed	제목은 작품 상세 읽기 상태 막대 그래프의 비율 표시 오류 수정이라는 주요 변경 사항을 명확하고 간결하게 설명합니다.
Description check	✅ Passed	이슈, 작업 설명, 변경 근거, 측정 결과가 포함되어 있습니다. 스크린샷 섹션에는 실제 이미지 대신 표가 있지만, 설명은 주요 변경 사항과 검토에 필요한 정보를 충분히 제공합니다.

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (3 skipped: 3 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/953

---

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.}

[devfeijoa]

수고하셨습니다!