Skip to content

SajuCard 잘림 현상 수정 - #149

Merged
Moony-H merged 1 commit into
developfrom
saju-card-bug-fix
Aug 19, 2026
Merged

SajuCard 잘림 현상 수정#149
Moony-H merged 1 commit into
developfrom
saju-card-bug-fix

Conversation

@Moony-H

@Moony-H Moony-H commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

관련 이슈

작업 내용

  • 마이페이지 사주 카드(SajuCard)가 일부 기기(좁은 화면 폭 / 시스템 글자 크기 확대)에서 텍스트가 잘려 보이는 현상 수정

변경사항 / 상세

  • SajuCardModifier.weight(1f) + aspectRatio(1f)로 화면 폭에 비례한 정사각형 박스를 만드는데, 내부 텍스트(한자 body1Bold 18sp + 읽는법 caption3Regular 10sp, 합산 lineHeight 39sp + 세로 padding 12dp)는 폭과 무관한 고정 sp 값이라 박스가 좁아지면 필요한 높이보다 작아져 clip(RoundedCornerShape)에 의해 잘리는 문제가 있었음
  • aspectRatio(1f) 앞에 heightIn(min = 64.dp)를 추가해, 폭이 좁아 정사각형 높이가 텍스트가 필요로 하는 최소 높이보다 작아지는 경우엔 세로로 살짝 늘어난 직사각형이 되도록 처리(폭이 충분할 때는 기존과 동일하게 정사각형 유지)

중점 리뷰사항

  • heightIn(min = 64.dp) 값이 실제 좁은 폭/큰 글자 설정 기기에서 충분한 여유인지 확인 부탁드립니다

스크린샷 (선택)

@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: e3de650f-b58e-4ce5-80c8-b7ab0e13a25a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@Moony-H Moony-H self-assigned this Aug 19, 2026
@Moony-H
Moony-H merged commit 0a13578 into develop Aug 19, 2026
3 checks passed
@Moony-H
Moony-H deleted the saju-card-bug-fix branch August 19, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant