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

[FE] Typography 템플릿 및 컴포넌트 구현 #53

Merged
merged 8 commits into from
Jul 14, 2023

Conversation

woo-jk
Copy link
Collaborator

@woo-jk woo-jk commented Jul 14, 2023

관련 이슈

closed #35

구현 기능 및 변경 사항

  • Figma Typography 팔레트 작성
  • Typography 컴포넌트 구현
  • Typography 컴포넌트 스토리북 구현

스크린샷(선택)

스크린샷 2023-07-14 오후 3 55 56

@woo-jk woo-jk added this to the 2차 스프린트 milestone Jul 14, 2023
@woo-jk woo-jk requested review from nlom0218 and yeopto July 14, 2023 06:58
@woo-jk woo-jk self-assigned this Jul 14, 2023
@woo-jk woo-jk linked an issue Jul 14, 2023 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@yeopto yeopto left a comment

Choose a reason for hiding this comment

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

Look so GODD😄

Copy link
Collaborator

@nlom0218 nlom0218 left a comment

Choose a reason for hiding this comment

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

GOOD 👍
코멘트 남겼어요!

type Story = StoryObj<typeof Typography>;

const meta: Meta<typeof Typography> = {
title: 'INPUTS/Typography',
Copy link
Collaborator

Choose a reason for hiding this comment

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

DATA DISPLAY 하위 폴더에 있으면 좋을 거 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

변경했습니다~

${({ variant, fontSize, fontWeight, color, $style }) => css`
font-size: ${fontSize || FONT_STYLE.fontSize[variant]};
font-weight: ${fontWeight || FONT_STYLE.fontWeight[variant]};
color: ${color || '#000'};
Copy link
Collaborator

Choose a reason for hiding this comment

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

#000 대신 이미 상수화 한 color 객체에서 black 불러와 사용하면 어떨까요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

변경했습니다!

@woo-jk woo-jk merged commit accf51b into develop Jul 14, 2023
1 of 2 checks passed
@woo-jk woo-jk deleted the fe/feature/35-Typography branch July 14, 2023 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] Typography 템플릿 및 컴포넌트 구현
3 participants