Conversation
원본 벡터 리소스(loading_indicator.xml)가 72x73dp(비정사각형)로 export돼 있어, 실제로는 원인 스피너가 세로로 살짝 긴 타원으로 렌더링되고 있었다. Figma 439:10407 프레임 자체는 74x74 정사각형이라 export 과정에서 생긴 오차로 보인다. 회전 애니메이션과 겹치며 미묘하게 중심이 어긋나 보이던 원인이라, 너비/높이 비율을 더 작은 쪽(72/74)으로 통일해 Image의 기본 ContentScale.Fit이 항상 정사각형(원)으로 렌더링하게 했다. - hilitLoadingIndicatorGeometry(): graphicWidth/graphicHeight를 같은 LOADING_INDICATOR_SIZE_RATIO(72/74)로 계산 - HilitLoadingIndicatorTest: 74dp 크기에서 72x72(기존 72x73), 148dp 크기에서 144x144(기존 144x146) 되도록 기대값 갱신 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
InterviewReportContent 의 Loading/Failed phase 가 CenteredMessage 에 Modifier.weight(1f) 만 넘기고 있었다. weight(1f) 는 Column 안에서 세로 공간만 분배할 뿐 가로 폭은 그대로 콘텐츠(스피너/문구)에 맞춰 줄어들고, Column 기본 정렬(Alignment.Start) 때문에 그 좁아진 Box 자체가 화면 왼쪽에 붙어버려 안의 Alignment.Center 가 화면 전체가 아니라 그 좁은 Box 안에서만 동작했다 — 그 결과 HilitLoadingIndicator/안내 문구가 화면 왼쪽으로 치우쳐 보였다(#176 실제 재현 화면). fillMaxWidth() 를 추가해 Box 가 화면 전체 폭을 차지하게 해서, 실제로 화면 가로 중앙에 스피너/문구가 오도록 고쳤다. 앞서 커밋한 HilitLoadingIndicator 그래픽 정사각형 수정과는 별개의, 이슈 재현 스크린샷에서 확인된 진짜 원인이다. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
|
Important Review available on request
Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 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 |
Contributor
Catalog Preview상태: ✅ 최신 배포 성공 Preview: https://yapp-github.github.io/28th-App-Team-1-Android/pr/200/ Commit: 실행: GitHub Actions |
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
1 participant
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.
🚩 연관 이슈
closed #176
📝 작업 내용
리포트 로딩/실패 화면에서 스피너·안내 문구가 화면 왼쪽으로 치우쳐 보이던 문제를 수정했습니다.
HilitLoadingIndicator그래픽 비율 정사각형 통일loading_indicator.xml)가 72x73(비정사각형)으로 export되어 스피너가 세로로 살짝 긴 타원처럼 보였던 문제 수정graphicWidth/graphicHeight를 동일 비율(72/74)로 계산하도록 변경, 관련 테스트 기대값 갱신InterviewReportContent의 Loading/Failed phase 정렬 수정CenteredMessage에Modifier.weight(1f)만 주고 있어,Column안에서 폭이 콘텐츠 크기로 줄어들고Column기본 정렬(Alignment.Start)에 따라 그 좁은 Box가 화면 왼쪽에 붙어버렸던 것이 실제 원인Modifier.weight(1f).fillMaxWidth()로 수정해 Box가 화면 전체 폭을 차지하도록 하여, 내부Alignment.Center가 실제 화면 가로 중앙 기준으로 동작하도록 수정🏞️ 스크린샷 (선택)
🗣️ 리뷰 요구사항 (선택)