Skip to content

fix(timeletter): 더보기 설정 아이콘을 gray5 토큰으로 칠해 대비를 맞춘다 (#2007) - #2318

Open
koongmai wants to merge 1 commit into
developfrom
fix/2007
Open

koongmai wants to merge 1 commit into
developfrom
fix/2007

Conversation

@koongmai

Copy link
Copy Markdown
Collaborator

📌𝘐𝘴𝘴𝘶𝘦𝘴

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

완료 조건 대응

#2007 완료 조건 이 PR
두 자리가 하드코딩 대신 gray5 토큰을 지나게 한다 TimeLetterBlockItem.kt · TimeletterListItem.kt 의 ⋯ 아이콘을 Image → Icon(painter = painterResource(R.drawable.setting), tint = AfternoteDesign.colors.gray5) 로 바꿨다. contentDescription·clickable·DropdownMenu 는 그대로다
라이트 대비 3:1 이상 실측 스크린샷 테스트 렌더에서 아이콘 픽셀을 직접 읽었다. 렌더된 아이콘 색은 #6E6E6E(= Gray5OnLight)로, #FAFAFA 위 4.89:1
다크에서도 보이는지 확인 계산만 했다. gray5 는 다크에서 #9E9E9E 그대로라 이번 변경으로 다크 값은 바뀌지 않고, 다크 배경 대비 6.01:1 이다. 다크 스크린샷 테스트가 없어 다크 렌더·실기기 확인은 하지 않았다
스크린샷 골든 갱신 screenshot-baseline 라벨로 CI가 갱신한다

이슈 본문과 다른 점

setting.xml 은 채움 #9E9E9E(2.57:1) 3곳 외에 테두리 #757575(4.41:1) 도 3곳 갖고 있다. 기존 골든에서 가장 진한 픽셀이 #757575 였던 이유다. Icon tint 는 채움·테두리를 모두 gray5 로 덮으므로 둘 다 4.89:1 이 된다. 드로어블 자체는 손대지 않았다.

검증 (로컬)

  • :feature:timeletter:presentation:validateDebugScreenshotTest — 차이가 나는 골든은 timeletterScreenList · Block · BlockCompact 3장이고, diff 이미지에서 바뀐 영역은 ⋯ 아이콘 2개뿐이다(Empty 는 변화 없음)
  • :feature:timeletter:presentation:testDebugUnitTest, ktlintCheck, :app:compileDebugKotlin 통과
  • core 변경 없음

🧪 CI Test Plan

{
  "androidTest": {
    "mode": "none",
    "reason": "변경이 feature/timeletter/presentation 의 두 컴포넌트에서 아이콘을 Image 에서 Icon(tint) 으로 바꾸는 렌더링에 한정되고 내비게이션·manifest·DI 를 건드리지 않는다. 결과는 스크린샷 테스트로 확인한다.",
    "tests": []
  }
}

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

screenshot-baseline 로 CI가 생성하는 골든 diff 참고 (⋯ 아이콘만 #6E6E6E 로 진해짐)

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

🤖 Generated with Claude Code

목록·블록 항목의 ⋯ 아이콘을 Image 대신 Icon(tint = gray5)으로 그린다.
드로어블 채움 #9E9E9E(2.57:1)·테두리 #757575(4.41:1)가 모두 gray5(#6E6E6E, 4.89:1)로 바뀐다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@koongmai koongmai added bug Something isn't working screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 area:timeletter 타임레터 기능 영역 labels Oct 26, 2026
@koongmai koongmai self-assigned this Oct 26, 2026

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

area:timeletter 타임레터 기능 영역 bug Something isn't working screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(timeletter): 「더보기 설정」 아이콘이 #9E9E9E 하드코딩으로 대비 2.57:1 (#1861 분리)

2 participants