-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feature: foundation 설정 #4
Merged
Merged
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
Thanks for the contribution! Please review the labels and make any necessary changes. |
와아 이야 대박..!!! 근데 components 하위에 shared 폴더는 무슨 역할일지 궁금합니더 |
나는 shared 폴더에 앱 내 공통으로 쓰이는 컴포넌트 정의한다고 생각했어! |
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.
👀 What is this PR?
foundation 설정을 했습니다 👍
typography + color + icon 설정했고 foundation 내 border-radius, divder는 따로 설정할건 없어서 제외했음!
📝 Changes
Typography
Pretendard는 subset font로 woff2 사용
font display 방식은 FOUT 정책으로
swap
으로 고정했어.tailwind classname 확장으로 font family, font size classname을 추가했는데 font-family랑 font-size를 합치진 못하더라합쳐서 사용할 수 있으면 좋긴한데 일단 당장엔 classname에 font랑 size 따로 세트로 명시해줘야해 ->
font-pretendard text-main-headline
typography 컴포넌트를 따로 만들까하다가 일단 button같은경우는 button 태그에 바로 스타일 입혀줘야하니깐 애매해서 보류했어
font는-> 이제 필요한 경우가 아니면 따로 명시해줄 필요는 없음!font-pretendard
,font-montserrat
이거 두가지 쓰면됨!typography를 이중에 사용하면 됨!
text-main-headline
,text-sub-headline
,text-body1
,text-body2
,text-body2-accent
,text-caption
,text-button1
,text-button2
,text-accent-eng
,text-body-eng
,text-caption-eng
개인적으로 naming 맘에 안들긴 한데(특히 영문일때) 일단 아래 사진대로 참고해서 지어놨어
더 좋은 방법 있으면 개선해보자 🙌
개선해서 합쳤어 👍 테일윈드 플러그인으로 component class를 정의할 수 있어서 새로운 컴포넌트 유틸리티 class를 추가했고 하나의 class에 글꼴 + size를 함께 정의해줬어
이제 요렇게 쓰면 돼 ->
text-{설정한 custom class명}
Color system
아직 grayscale 밖에 없어서 이렇게 설정했어
classname 예시:
bg-primary
,bg-grayscale-100
,text-primary
fill-grayscale-700
Icon
icon을 사용하기 쉽게 Icon 컴포넌트를 구현했어
Icon 사용법
이런식으로 힌트가 뜹니닷
width
,height
는 right arrow 제외하고 모두 default 24인데 right arrow일 때에만(이건 가로가 더 길더라)width
만 default 40으로 setting기타
일단 type, component, folder 컨벤션 내가 편한대로 하긴했는데 이대로 따라가도 좋고 맞춰도 좋고! 🙌
📌 Related issue(s)
📷 Attachment(optional)