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

현재 모임 상태 태그구현 #142

Merged
merged 5 commits into from
Jul 31, 2024
Merged

현재 모임 상태 태그구현 #142

merged 5 commits into from
Jul 31, 2024

Conversation

jaeml06
Copy link
Contributor

@jaeml06 jaeml06 commented Jul 30, 2024

PR의 목적이 무엇인가요?

태그 컴포넌트 구현

props로 color를 받아 ( 'red' | 'green' | 'gray') 색상을 변경할 수 있는 태그 컴포넌트 구현

이슈 ID는 무엇인가요?

설명

image image image

질문 혹은 공유 사항 (Optional)

@jaeml06 jaeml06 added FE 프론트엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) labels Jul 30, 2024
@jaeml06 jaeml06 requested review from ss0526100 and cys4585 July 30, 2024 07:27
@jaeml06 jaeml06 changed the title Feat/#140 현재 모임 상태 태그구현 Jul 30, 2024
Copy link
Contributor

@cys4585 cys4585 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 :)

Comment on lines +168 to +174
tag: css`
font-family: Pretendard;
font-size: 1.2rem;
font-style: normal;
font-weight: 600;
line-height: 130%;
`,
Copy link
Contributor

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 = {
Copy link
Contributor

Choose a reason for hiding this comment

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

정말 사소하지만 GreenTag, RedTag와 같이 색상을 표현하는 것보다 의미를 표현하면 어떨까요? (너무 사소한가?ㅋㅋ)

Copy link
Contributor

@ss0526100 ss0526100 left a comment

Choose a reason for hiding this comment

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

저는 태그가 애매하게 느껴지는 경우가 많더라구요

ex)넓이가 고정된 태그vs넓이가 고정되어 있지 않는 태그

일단 컴포넌트화가 이루어졌다면 어프루브 드리는게 좋을 것 같아요

@ss0526100 ss0526100 merged commit afe315f into develop-frontend Jul 31, 2024
1 check passed
@ss0526100 ss0526100 deleted the feat/#140 branch July 31, 2024 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants