You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PR #1816 의 승인 리뷰가 [nit] 로 남기고 간 잔여 2건이다. 이 둘을 추적하는 열린 이슈가 없어 새로 연다. 기준 커밋은 origin/develop86ab223.
타이포 동일성 잠금이 여섯 필드만 본다
core/ui/src/test/kotlin/com/afternote/core/ui/theme/AfternoteTypographyPairTest.kt 의 comparedFields() 와 assertSameStyle() 이 fontFamily·fontWeight·fontSize·lineHeight·lineBreak·localeList 여섯 개만 비교한다. bodySmallB/bodySmallR 나 captionLargeB/captionLargeR 한쪽에만 letterSpacing·color 같은 나머지 필드를 넣으면 두 자리가 실제로 갈라져도 잠금을 그냥 빠져나간다. assertEquals(r, b) 로 TextStyle 전체를 비교하면 그 틈이 닫힌다. 지금 두 쌍은 나머지 필드가 전부 기본값이라 비교 범위를 넓혀도 실패하지 않는다. 렌더 변화가 없으니 골든 재생성도 필요 없다.
Bold 하한을 적은 KDoc 이 사실과 다르다
core/ui/src/main/kotlin/com/afternote/core/ui/theme/AfternoteTypography.kt 88~90행이다. 하한을 "기본 설정(밀도 480 · 글꼴 1.0)에서 약 19sp, 글꼴 배율을 0.8 로 줄인 기기에서는 약 28sp" 로 적고 "이 파일의 스타일은 전부 그 아래라 Bold 를 쓸 자리가 없다" 로 닫는데, 같은 파일의 h1 은 28sp, h2 는 24sp, h3 는 20sp 라 셋 다 기본 설정 하한 위다. "전부 그 아래" 는 위 표를 잰 기기(밀도 420 · 글꼴 배율 0.8, sp→px 2.1)에서만 성립하고 그때도 h1 은 58.8px 로 58px 하한을 넘긴다. 제목에 Bold 가 없는 실제 근거는 획 대비가 아니라 시안이고, 그 근거는 바로 다음 문단에 이미 있다. 그 한 문단만 조건부로 고친다.
완료 조건은 셋이다. assertSameStyle() 이 TextStyle 전체를 비교하고, 강화 전에는 초록이던 한쪽 letterSpacing 변이가 강화 후에는 실패하는 것을 실측으로 확인하며, KDoc 이 어느 설정에서 어느 스타일이 하한 위인지 정확히 적는다. 검증은 :core:ui:testDebugUnitTest.
이 둘은 타이포 B/R 토큰의 유지·통합 판정(#1862, PM 대기)과 무관하다. 값을 바꾸지 않고 "두 쌍이 같다" 는 오늘의 계약을 잠그는 구멍을 메우는 것이고, KDoc 쪽은 #1815 의 획 대비 분석에 딸린 서술 오류라 B/R 이름 문제와 축이 다르다. #1862 가 유지로 끝나면 잠금이 그대로 남고, 통합으로 끝나면 잠금이 테스트와 함께 사라진다. 어느 쪽이든 이 변경이 걸림돌이 되지 않는다.
Activity