Skip to content

fix(setting): 수신자 아바타·배지 시안 정합 (#2003) - #2293

Open
1hyok wants to merge 2 commits into
developfrom
fix/2003-avatar-geometry
Open

1hyok wants to merge 2 commits into
developfrom
fix/2003-avatar-geometry

Conversation

@1hyok

@1hyok 1hyok commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

먼저 볼 파일

  • ProfilePhotoWithAddBadge.kt: 수신자 등록·수정 공용 아바타와 장식 배지
  • setting_ic_plus.xml: 참조가 사라진 배지 벡터 삭제

변경

수신자 등록·수정의 표시 전용 아바타를 기존 core ProfileImage와 PlusBadgeButton으로 조합했습니다. 기본 실루엣과 배지의 48dp 원, 17dp 내부 여백을 core 프로필과 맞춥니다. 배지는 클릭 없이 표시하며 사진 선택 콜백을 추가하지 않습니다. 이 컴포넌트는 setting 모듈에서만 소비하므로 internal로 제한했습니다.

프로필 수정은 #2152에서 이미 전환됐습니다. 이번 변경은 수신자 공용 표시 영역과 사용처가 사라진 배지 자산만 다룹니다. 설정 첫 화면이 사용하는 기본 아바타는 유지합니다.

선행 관계

기준 develop: 929fa8ca73233d4ffae6b1e6ca5ca85c4efa0300. 수신자 수정 화면도 ReceiverRegisterContent를 소비합니다. #944의 등록 API·상태·내비게이션 전환을 소비하거나 변경하지 않으며, 공용 표시 컴포넌트를 교체한다는 확정 근거가 없어 독립 범위로 진행했습니다.

검증

  • :feature:setting:presentation:compileDebugKotlin, :feature:setting:presentation:ktlintCheck 통과
  • 실제 base 전체 diff의 모듈 소유권 검사와 재작성 사전점검 통과
  • Kotlin LSP documentSymbol에서 변경 컴포넌트 확인
  • CI 표준 환경에서 일반·compact Preview baseline 2개 생성·검증 및 적용 완료 (a0e08c76a300076ec282a6a9dd1f54f359cb41e3). 한글·입력 배치와 아바타·배지 기하를 눈으로 확인했습니다.
  • 생성 실행. 최신 HEAD 필수 검사는 진행 중입니다.

화면 비교

기존 develop baseline과 CI에서 생성한 이번 변경 baseline입니다.

변경 전 변경 후
변경 전 변경 후

CI Test Plan

{
  "androidTest": {
    "mode": "none",
    "reason": "표시 전용 아바타의 자산과 배지 기하만 변경한다. 상태·이벤트·사진 선택·내비게이션은 변경하지 않으며 기존 일반·compact Compose Preview 스크린샷으로 시각 범위를 검증한다.",
    "tests": []
  }
}

Closes #2003

@github-actions github-actions Bot added the android-test PR head에서 Android Managed Device 계측 테스트 실행 label Oct 7, 2026
@1hyok 1hyok added the screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 label Oct 7, 2026
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

📸 CI 표준 환경에서 screenshot baseline 2개를 생성·검증해 a0e08c7 로 커밋했습니다.
실행 로그·생성 artifact: https://github.com/Afternote/Afternote-FE/actions/runs/37591847062
이미지 내용은 이 PR의 PNG diff에서 눈으로 최종 확인해 주세요. 필수 검사도 새 SHA에 다시 요청했습니다.

@github-actions github-actions Bot removed the screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 label Oct 7, 2026
@1hyok

1hyok commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator Author

최신 HEAD a0e08c76a300076ec282a6a9dd1f54f359cb41e3의 검증이 완료됐습니다.

전후 픽셀 비교에서 일반 화면은 (364,168)-(716,520), compact 화면은 (226,128)-(494,396)의 아바타·배지 영역만 변경됐습니다. 이미지 크기·한글·입력 영역은 유지됩니다. Viewed는 3/4이며 구현 파일의 사용자 확인을 기다립니다.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

android-test PR head에서 Android Managed Device 계측 테스트 실행

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(setting): 프로필 사진 자리가 앱 나머지와 다른 자산·배지로 그려진다 (#1782 후속)

2 participants