-
Notifications
You must be signed in to change notification settings - Fork 7
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
현재 모임 상태 태그구현 #142
현재 모임 상태 태그구현 #142
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다 :)
tag: css` | ||
font-family: Pretendard; | ||
font-size: 1.2rem; | ||
font-style: normal; | ||
font-weight: 600; | ||
line-height: 130%; | ||
`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
figma 디자인 시스템 typography에는 아직 tag에 대한 것은 적용되어 있지 않은 것 같아요!
@ksk0605 테바에게 공유가 되어야 할 것 같아요~
이번 케이스와 같이 개발을 하면서 필요할 것 같은 규칙이 있다면 추가하고 공유해드리면 될까요?
render: (args) => <Tag {...args} />, | ||
}; | ||
|
||
export const RedTag: Story = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
정말 사소하지만 GreenTag, RedTag와 같이 색상을 표현하는 것보다 의미를 표현하면 어떨까요? (너무 사소한가?ㅋㅋ)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저는 태그가 애매하게 느껴지는 경우가 많더라구요
ex)넓이가 고정된 태그vs넓이가 고정되어 있지 않는 태그
일단 컴포넌트화가 이루어졌다면 어프루브 드리는게 좋을 것 같아요
PR의 목적이 무엇인가요?
태그 컴포넌트 구현
props로 color를 받아 ( 'red' | 'green' | 'gray') 색상을 변경할 수 있는 태그 컴포넌트 구현
이슈 ID는 무엇인가요?
설명
질문 혹은 공유 사항 (Optional)