Repository navigation
[Fix] Paperlogy 굵기별 폰트 매핑 안정화 - #92
Open
lioba00700 wants to merge 1 commit into
Open
lioba00700 wants to merge 1 commit into
lioba00700 wants to merge 1 commit into
Conversation
Contributor
|
Warning Review limit reachedThis review ran on the open-source allowance, not this organization's plan, because the pull request author doesn't have an assigned seat. Waiting won't change this — ask an organization admin to assign them a seat, or add seats in Billing if every seat is already assigned, then retry. Next included review available in 59 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (48)
Comment |
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
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.
🎯 작업 내용
Paperlogy 정적 폰트 9개 굵기가 NativeWind/Tailwind의 기본 font-weight 유틸리티와 충돌하지 않도록 font-family 매핑을 전면 정리했습니다.
공용 UI 컴포넌트와 화면의 기존 굵기 클래스를 새 Paperlogy family 유틸리티로 이관하고, 같은 충돌 이름이 다시 도입되지 않도록 회귀 테스트를 추가했습니다.
🔍 기존 문제
기존 설정은
--font-bold,--font-semibold처럼 Tailwind가 기본 font-weight 용도로 사용하는 이름을 Paperlogy font-family 토큰에도 사용했습니다.이전 수정은 공용 컴포넌트에서
font-sans를 제거해 당시 클래스 우선순위 문제를 완화했지만 이름 충돌 자체는 남아 있었습니다. 따라서 컴파일 순서, 캐시, 컴포넌트별 기본 클래스 조합에 따라 Paperlogy 굵기가 다시 풀릴 수 있었습니다.✅ 주요 변경 사항
충돌 없는 Paperlogy 토큰
font-paperlogy-thin부터font-paperlogy-black까지 9개 family 유틸리티 정의font-bold,font-medium등과 이름 공간 분리공용 UI 및 화면 이관
회귀 방지
test:typography스크립트 추가🧪 확인 사항
📌 참고 사항
Expo Font의 config plugin과 runtime
useFonts로딩 구조는 변경하지 않았습니다. 이번 PR은 로딩 방식이 아니라 NativeWind font-family 선택 계약만 수정합니다.웹 export는 기존
react-native-pager-view의 native-only import 문제로 실패하며 이번 폰트 변경과는 무관합니다. iOS 대상 Metro 번들은 정상 생성됐습니다.