Repository navigation
fix(mindrecord): MEMORY SPACE 헤더 두 문구의 행간을 시안 150% 에 맞춘다 (#1487) - #1547
Conversation
`copy(fontSize = …)` 는 **lineHeight 를 원 토큰 값 그대로 남긴다.** 부제가 9sp 글자에 `inter` 토큰의 21sp 행간을 그대로 이어받아 행간 배율이 **233%** 였다 (#1487). 제목은 덮지도 않아 토큰 13/21 을 그대로 썼다. 시안(정리 Screen Design › 홈 › 기억공간)은 둘 다 150% 다. | | 시안 | 종전 | 지금 | |---|---|---|---| | 제목 `MEMORY SPACE` (`4327:67831`) | 11 / 16.5 | 13 / 21 | 11 / 16.5 | | 부제 (`4327:67833`) | 9 / 13.5 | 9 / **21** | 9 / 13.5 | 제목은 이슈가 「범위 밖, 판단은 담당자 몫」으로 넘긴 건이다. 같은 컴포저블의 인접 3줄이고 시안 노드로 값이 확정돼 함께 뗐다 — 나누면 같은 자리를 두 번 고치고 골든도 두 번 재생성한다. 색(부제 `gray5` vs 시안 `gray6`)과 자간(부호가 반대)은 판정 축이 달라 건드리지 않았다. 근거는 `docs/qa/assumptions.md` 에 적었다. 스크린샷 골든 7건이 이 변경으로 어긋난다. macOS 로컬 재생성은 무관 골든까지 틀어지므로 `screenshot-baseline` 라벨로 CI 에서 재생성한다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
🚧 리뷰 적체 가드 — 이 PR 을 닫습니다리뷰 응답을 기다리는 PR 이 1건 남아 있습니다.
이 저장소는 리뷰 응답을 기다리는 PR 이 1건이라도 남아 있으면 새 PR 을 열 수 없습니다 (#1136). 각 PR 에 필요한 리뷰는 한 건입니다 — 누구든 먼저 판정을 내면 그 PR 은 모두의 목록에서 빠집니다. 승인 뒤 변경을 다시 볼 의무는 없고, 변경요청도 작성자가 같은 리뷰어에게 재요청한 뒤 반영 커밋을 올린 경우에만 다시 리뷰어의 몫이 됩니다. 되살리는 법위 리뷰를 끝낸 뒤 이 PR 을 다시 여세요. 브랜치는 지워지지 않았고 커밋도 그대로입니다. gh pr reopen 1547리뷰보다 이 PR 이 급하면 |
🚧 리뷰 적체 가드 — 이 PR 을 닫습니다리뷰 응답을 기다리는 PR 이 1건 남아 있습니다.
이 저장소는 리뷰 응답을 기다리는 PR 이 1건이라도 남아 있으면 새 PR 을 열 수 없습니다 (#1136). 각 PR 에 필요한 리뷰는 한 건입니다 — 누구든 먼저 판정을 내면 그 PR 은 모두의 목록에서 빠집니다. 승인 뒤 변경을 다시 볼 의무는 없고, 변경요청도 작성자가 같은 리뷰어에게 재요청한 뒤 반영 커밋을 올린 경우에만 다시 리뷰어의 몫이 됩니다. 되살리는 법위 리뷰를 끝낸 뒤 이 PR 을 다시 여세요. 브랜치는 지워지지 않았고 커밋도 그대로입니다. gh pr reopen 1547리뷰보다 이 PR 이 급하면 |
|
📸 CI 표준 환경에서 screenshot baseline 7개를 생성·검증해 |
OnlyReadOneBook
left a comment
There was a problem hiding this comment.
승인합니다. 시안 값을 Figma REST 로 독립 조회해 두 노드 모두 PR 값과 일치하는 것을 확인했습니다.
시안 교차 검증 (2026-08-30, Figma REST 직접 조회)
| 노드 | name | fontSize | lineHeightPx | 배율 | letterSpacing |
|---|---|---|---|---|---|
4327:67831 |
MEMORY SPACE |
11.0 | 16.5 | 150% | 0.6145 |
4327:67833 |
마우스를 움직여 공간을 탐색하세요 |
9.0 | 13.5 | 150% | 0.1670 |
PR 본문·KDoc·assumptions.md 에 적힌 값이 전부 이와 같습니다. 부제 값은 제가 #1487 을 낼 때 08-29 에 실측한 것과도 일치하고, 제목(11/16.5)은 이 PR 이 새로 조회한 값이라 이번에 독립 확인했습니다 — 맞습니다.
범위를 제목까지 넓힌 판단
이슈가 「범위 밖, 판단은 담당자 몫」 으로 넘긴 것을 한 PR 로 묶은 근거(같은 컴포저블 인접 3줄 · 시안 노드로 값 확정 · 나누면 골든을 두 번 재생성)에 동의합니다. 제목은 copy 조차 없이 토큰 그대로(13/21)여서 부제와 같은 뿌리이고, 실제로 부제만 고치면 제목이 여전히 어긋난 채 남아 「행간을 시안에 맞췄다」 가 반만 참이 됩니다.
색·자간을 건드리지 않고 assumptions.md 에 남긴 것도 맞는 판단입니다. 부제 색(시안 #757575=gray6 vs 코드 gray5)과 자간 부호 반대(시안 +0.019em/+0.056em vs 토큰 −0.006em)는 판정 축이 달라(대비·가독성 / 토큰 전반) 같이 바꾸면 리뷰가 세 축을 한꺼번에 봐야 합니다. #1403 → PR #1442 선례와도 정합합니다.
골든 재생성 경위
github-actions[bot] 의 test: update screenshot baselines in CI 커밋으로 CI 컨테이너에서 생성됐습니다. 이 레포의 골든 정본이 컨테이너라 로컬(macOS) 재생성은 무관 골든까지 틀어지는데, 그 함정을 피했습니다. 재생성 범위도 MemorySpaceScreenScreenshotTestKt · MemorySpaceHeaderLargeFontScreenshotTestKt 두 클래스뿐이라 헤더를 포함하는 화면에만 국한됩니다 — 무관 골든 오염 없음.
픽셀 계측(종전 y=71..143 73px 한 덩어리 → 지금 y=70..90 제목 + y=97..137 부제, 사이 6px 여백)으로 두 줄이 붙어 구분되지 않던 것이 분리된 실제 효과까지 보인 점이 좋습니다.
검증
| 항목 | 결과 |
|---|---|
Figma REST 노드 조회 (4327:67831 · 4327:67833) |
✅ PR 값과 일치 |
| CI | 실패·대기 체크 0건 |
| 골든 생성 경위 | ✅ CI 봇 (컨테이너 정본) |
| 골든 재생성 범위 | ✅ 헤더 포함 화면 2개 클래스만 |
| QA | 별도 실기 QA 없이 위 시안 조회와 PR 의 픽셀 계측으로 갈음했습니다 — 판정 축이 정적 타이포 값이라 스크린샷 골든이 실기보다 촘촘합니다 |
참고로 이 건은 제 #1444(「타이포 토큰을 fontSize 만 copy 해 행간이 어긋나는 3곳」)에서 갈라진 mindrecord 몫입니다. 남은 두 자리는 NumberKeypad.kt:80(setting) 과 SenderDetailScreen.kt:175(receiver) 입니다.
…ace-header-typography
Closes #1487
무엇을
copy(fontSize = …)는lineHeight를 원 토큰 값 그대로 남긴다.MemorySpaceHeader의 두 문구가 그래서 어긋나 있었습니다.MEMORY SPACE시안 대조
정본 「정리 Screen Design」 › 홈 › 기억공간에서 직접 조회했습니다.
4327:67831제목text-[11px] leading-[16.5px]4327:67833부제text-[9px] leading-[13.5px]부제 문구는 시안 「마우스를 움직여 공간을 탐색하세요」 / 코드 「화면을 드래그해 공간을 둘러보세요」로 다르지만 같은 자리입니다(모바일 각색).
제목을 함께 고친 이유
이슈가 제목을 「범위 밖, 판단은 담당자 몫」으로 넘겼습니다. 같은 컴포저블의 인접 3줄이고 시안 노드로 값이 확정돼 한 PR 로 묶었습니다 — 나누면 같은 자리를 두 번 고치고 스크린샷 골든도 두 번 재생성해야 합니다.
시연
memorySpaceScreenScreenshot렌더를 골든과 겹쳐 봤습니다. 위가 종전, 아래가 지금입니다.diff 이미지에서 바뀐 픽셀이 이 두 줄뿐인 것도 확인했습니다(화면의 나머지는 전부 동일).
픽셀로도 셌습니다 — 헤더 상단의 «잉크가 있는 행» 구간입니다.
건드리지 않은 것
같은 노드 조회에서 두 가지가 더 어긋납니다. 판정 축이 달라 별도로 뗐습니다 — 여기서 함께 바꾸면 리뷰가 세 축을 한꺼번에 봐야 합니다.
#757575(=gray6)gray5(#9E9E9E)+0.167px· 제목+0.6145px-0.006em— 부호가 반대선례(#1403 → PR #1442)도
fontSize+lineHeight만 다뤘습니다. 근거는docs/qa/assumptions.md에 적었습니다. 후속 이슈로 올리겠습니다.검증
:feature:mindrecord:presentation:compileDebugKotlin:feature:mindrecord:presentation:ktlintCheckCI Test Plan
{ "androidTest": { "mode": "none", "reason": "타이포 토큰 두 줄의 fontSize·lineHeight 변경이다. 동작·내비게이션·네트워크 경로가 바뀌지 않아 계측이 판별할 것이 없다. 시각 회귀는 이 모듈의 screenshot 골든이 덮으며, 이 PR 은 그 골든 7건을 CI 에서 재생성한다." } }