feat: 컬러, 폰트 시스템 구성#21
Conversation
Walkthrough이 변경 사항은 폰트 스타일을 위한 새로운 CSS 파일을 추가하고, 메인 CSS에 해당 폰트 스타일을 임포트합니다. 또한 글로벌 스타일 모듈에 디자인 토큰 시스템을 도입하여 색상, 간격, 반응형 브레이크포인트, 타이포그래피 등 다양한 스타일 속성을 구조화된 객체와 CSS 커스텀 프로퍼티로 정의합니다. Changes
Poem
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/styles/fonts.css (1)
11-81: Paperlogy 폰트 최적화 제안: 단일 Variable/WOFF2 사용 검토 필요
- Pretendard Variable(woff2) 파일: src/assets/fonts/PretendardVariable.woff2 확인됨
- Paperlogy TTF 파일(9개): src/assets/fonts/Paperlogy-1Thin.ttf … Paperlogy-9Black.ttf 확인됨
→ 네트워크 요청 수와 로딩 최적화를 위해 Paperlogy도 단일 Variable 폰트(.woff2) 또는 WOFF2 버전으로 제공 가능한지 확인해 주세요.
src/styles/global/global.ts (1)
122-131: 브레이크포인트 구조 개선을 제안합니다현재 구조는 mobile 객체와 tablet 단일 값이 혼재되어 일관성이 부족합니다.
다음과 같이 구조를 통일하는 것을 권장합니다:
export const breakpoints = { mobile: { min: '320px', standard: '360px', max: '767px', }, - tablet: '768px', + tablet: { + min: '768px', + max: '1023px', + }, + desktop: { + min: '1024px', + }, };
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (10)
src/assets/fonts/Paperlogy-1Thin.ttfis excluded by!**/*.ttfsrc/assets/fonts/Paperlogy-2ExtraLight.ttfis excluded by!**/*.ttfsrc/assets/fonts/Paperlogy-3Light.ttfis excluded by!**/*.ttfsrc/assets/fonts/Paperlogy-4Regular.ttfis excluded by!**/*.ttfsrc/assets/fonts/Paperlogy-5Medium.ttfis excluded by!**/*.ttfsrc/assets/fonts/Paperlogy-6SemiBold.ttfis excluded by!**/*.ttfsrc/assets/fonts/Paperlogy-7Bold.ttfis excluded by!**/*.ttfsrc/assets/fonts/Paperlogy-8ExtraBold.ttfis excluded by!**/*.ttfsrc/assets/fonts/Paperlogy-9Black.ttfis excluded by!**/*.ttfsrc/assets/fonts/PretendardVariable.woff2is excluded by!**/*.woff2
📒 Files selected for processing (3)
src/main.css(1 hunks)src/styles/fonts.css(1 hunks)src/styles/global/global.ts(1 hunks)
🔇 Additional comments (7)
src/styles/fonts.css (2)
84-101: 포괄적인 폰트 폴백 스택이 잘 구성되었습니다한국어 사이트에 적합한 시스템 폰트들이 적절히 포함되어 있으며, 이모지 폰트까지 고려된 점이 훌륭합니다.
103-115: 폰트 유틸리티 클래스의 용도를 명확히 해주세요
.font-lock클래스가 monospace 폴백을 사용하는 이유와 구체적인 사용 목적이 불분명합니다. 또한.font-secondary가 serif 폴백을 사용하는 것이 의도된 것인지 확인이 필요합니다.폰트 클래스들의 사용 목적을 문서화하거나 주석으로 명시해주시기 바랍니다.
src/main.css (1)
2-2: 폰트 스타일시트 임포트가 적절히 추가되었습니다reset.css 이후에 폰트 정의를 로드하는 순서가 올바릅니다.
src/styles/global/global.ts (4)
93-97: 타이포그래피 시스템과 fonts.css의 일관성을 확인했습니다폰트 패밀리 정의가 fonts.css의 폰트 선언과 일치하여 좋습니다.
232-369: CSS 커스텀 프로퍼티 시스템이 포괄적으로 구성되었습니다디자인 토큰을 CSS 변수로 변환하는 방식이 체계적이며, 반응형 브레이크포인트별 변수 설정도 잘 구현되었습니다.
354-368: 반응형 변수의 용도를 명확히 해주세요
--responsive-margin,--responsive-gutter,--responsive-columns변수들이 정의되어 있지만, 이들이 어떻게 사용될지 명확하지 않습니다. 특히 모바일과 태블릿에서 동일한 margin 값(20px)을 사용하는 이유를 확인해주세요.이러한 반응형 변수들의 사용 목적과 값 설정 근거를 문서화해주시기 바랍니다.
371-379: 글로벌 스타일과 디자인 토큰 통합이 우수합니다typography.fontFamily.primary를 사용하여 일관성을 유지하고, designTokens CSS를 적절히 주입한 점이 훌륭합니다.
📝작업 내용
Summary by CodeRabbit
New Features
Style