Skip to content
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

chore: 피그마에 정의된 Typography 디자인 시스템 적용 #26

Merged
merged 5 commits into from
Jul 21, 2024

Conversation

hjy0951
Copy link
Collaborator

@hjy0951 hjy0951 commented Jul 18, 2024

🤔 어떤 문제가 발생했나요?

  • 피그마에 정리된 디자인 시스템을 적용하기 위해서는 텍스트 관련 모든 속성을 하나씩 정의해주어야 한다.
  • 피그마에 존재하는 font-weight: regular 속성이 정의되어 있지 않다.

🎉 어떻게 해결했나요?

  • Pretendard 폰트를 variable 파일을 이용하여 적용하였습니다.
  • PandaCSS의 textStyle로 디자인 시스템을 정의하였습니다.
  • config 파일에 fontWeight: regular 속성을 정의하였습니다.

📷 이미지 첨부 (Option)

  • 폰트 적용도 잘되었고
스크린샷 2024-07-18 오후 5 19 58
  • 디자인 시스템 적용도 잘되고
스크린샷 2024-07-18 오후 5 20 23
  • 자동 완성도 잘됩니다~
스크린샷 2024-07-18 오후 6 56 21

⚠️ 유의할 점! (Option)

  • 디자인 시스템에 명칭에 따라 weight를 다르게 가져갈 수 있어서 textStyle로 명칭을, fontWeight로 굵기를 설정해주어야 합니다. 기본 사용법은 아래와 같습니다!
<p className={css({ textStyle: 'display1', fontWeight: 'bold' })}>
  This is a paragraph from Panda with the body text style.
</p>
  • 속성 두가지를 이름까지 한번에 정의하는 것이 조금 번거로울 수도 있다는 생각에 아래와 같은 함수를 추가할까 했습니다.
  • 하지만 자동 완성이 가져다 주는 이점도 있고, 그렇게 번거로울 것까지 있나 싶은 생각에 우선 반영하지는 않았는데 다른 분들의 의견이 궁금합니다~
/* function */
const textCss = (textStyle: string, fontWeight: string = 'medium') => {
  return css({ textStyle, fontWeight });
};

/* 사용 예시 */
<p className={textCss('body1.normal', 'regular')}>
  This is a paragraph from Panda with the body text style.
</p>

@king2dwellsang king2dwellsang added the Review Plz🙏 # Review is not yet complete label Jul 18, 2024
@king2dwellsang king2dwellsang added chore # Chores style # Style code (code formatting) labels Jul 18, 2024
Copy link
Member

@wokbjso wokbjso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍👍👍👍

저는 font-weight 속성 하나만 따로 지정하는거는 유틸 함수 없이 따로 지정해줘도 괜찮을 것 같습니다~!!

@king2dwellsang king2dwellsang added Approved 🆗 # Review is completed and removed Review Plz🙏 # Review is not yet complete labels Jul 18, 2024
Copy link
Member

@Jungjjeong Jungjjeong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍👍👍👍👍

함수로 만들어서 사용하는 것에 대한 명확한 이점이 없을듯!
확장성을 위해서라도 따로 넣어서 사용하는 것이 좋을 것 같습니다~!

@hjy0951 hjy0951 merged commit 5153601 into main Jul 21, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved 🆗 # Review is completed chore # Chores style # Style code (code formatting)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants