[Feature] Color, Typo, Spacing 페이지 마크업 작성 #187
Conversation
This file contains 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
- sub에 string 뿐만 아니라 ReactNode가 올 수 있도록 수정했어요. - 이미지가 좌측 정렬된 경우가 있어서 ImageAlign을 추가했어요.
soulchicken
requested review from
eugene028,
hamo-o and
SeieunYoo
as code owners
|
|
Size Change: 0 B Total Size: 680 kB ℹ️ View Unchanged
|
SeieunYoo
reviewed
Jan 8, 2025
There was a problem hiding this comment.
수고 많으셨어요~~ 남겨주신 코멘트에 대해 먼저 리뷰 남겨드리자면..!!
roboto mono 폰트 파일을 추가한 이후에 packages/wow-tokens/src/typography.ts에만 반영했는데, 화면에는 반영되지 않았습니다. 이후에 어떤 조치를 해야하나요? 폰트나 typo같은 내용이 컴포넌트에 적용되기까지 과정이 궁금해요.
- wow-token 에 타이포 추가해주신 이후에 wow-theme/tokens/typography 쪽에서도 추가해주시면 됩니다!
wow-theme 에서 정의한 style 이 (e.g. textStyle)이 panda config 에 들어갈 preset 으로 만들어져 저희가 정의한 컬러나 타이포가 적용됩니다. wow-theme 패키지는 공통된 panda config 를 관리하기 위해 만들어졌어요. wow-theme
기존에 구현된 packages/wow-tokens/src/typography.ts의 label1U에도 위의 1과 같은 궁금증이 있어요. textDecoration: "underline"가 들어갔지만 제 화면에는 랜더링되지 않는바람에 더 궁금합니다. ㅠㅠ
- 살펴보니까 제가 wow-theme 에서 label1U와 label2U 의 value 값을 잘못 정의했더라구요...! 그래서 아마 underline 이 적용되지 않은 거 같아요.
wow-theme 에서 label1U, label2U, label2U2 에 올바른 value 값이 들어갈 수 있도록 조정해주실 수 있을까요?
wow-theme/tokens/typography
as-is
label1U: {
value: typography.label1,
},
to-be
label1U: {
value: typography.label1U,
},
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.
🎉 변경 사항
Foundation의 Color, Typography, Spacing 페이지의 마크업을 작성했어요.
유진님의 PR(#185)처럼 현영님의 컴포넌트를 사용했어요.
🚩 관련 이슈
🙏 여기는 꼭 봐주세요!
ImageCards 컴포넌트에 있는 Card의 CSS를 바꿔야해서 Item Type을 수정하고 내려준 상태입니다. 리뷰할 때 확인해서 피드백 부탁드려용...
[해결]
typography의 label을 구현하는 과정에서 해결하지 못해서 궁금한 점이 2가지가 생겼어요.
feat: Roboto Mono 폰트 추가, feat: label2_mono 토큰 추가이 커밋처럼 작업을 한 상태입니다.
roboto mono 폰트 파일을 추가한 이후에packages/wow-tokens/src/typography.ts에만 반영했는데, 화면에는 반영되지 않았습니다. 이후에 어떤 조치를 해야하나요? 폰트나 typo같은 내용이 컴포넌트에 적용되기까지 과정이 궁금해요.기존에 구현된packages/wow-tokens/src/typography.ts의label1U에도 위의 1과 같은 궁금증이 있어요.textDecoration: "underline"가 들어갔지만 제 화면에는 랜더링되지 않는바람에 더 궁금합니다. ㅠㅠ