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] feat: 카테고리에 따른 태그 색상 변경 #359

Merged
merged 9 commits into from
Aug 11, 2023

Conversation

hae-on
Copy link
Collaborator

@hae-on hae-on commented Aug 9, 2023

Issue

✨ 구현한 기능

카테고리에 따라 태그 색상을 변경할 수 있도록 하였습니다.

📢 논의하고 싶은 내용

x

🎸 기타

x

⏰ 일정

  • 추정 시간 : 2시간
  • 걸린 시간 : 2시간

Copy link
Collaborator

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

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

수고했어옹 해옹

Copy link
Collaborator

@Leejin-Yang Leejin-Yang left a comment

Choose a reason for hiding this comment

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

해온 수고했어요 👍
간단하게 의견 남겼는데 코멘트 확인해주세요!

@@ -21,6 +25,7 @@ const ReviewTagItem = ({ id, name, isSelected }: ReviewTagItemProps) => {
<Button type="button" weight="bold" variant="transparent" onClick={handleReviewTag}>
<TagBadge
isSelected={isSelected}
tagColor={tagColor}
Copy link
Collaborator

Choose a reason for hiding this comment

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

variant를 받아서 스타일드에서 스타일로 넣어두는건 어떤가요?
디자인 시스템에서 했던 느낌으로다가요

const tagColorStyles = {
  TASTE: css`
    border: ...
    color: ...
  ...
}

아마 이런 느낌??

@github-actions
Copy link

Unit Test Results

2 tests   2 ✔️  6s ⏱️
1 suites  0 💤
1 files    0

Results for commit 01b8d76.

Copy link
Collaborator

@Leejin-Yang Leejin-Yang left a comment

Choose a reason for hiding this comment

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

유틸의 파일만 지우면 될거 같아요
수고했어요 해온 🌞♨️

@hae-on hae-on merged commit ffb6d53 into develop Aug 11, 2023
@hae-on hae-on deleted the feat/issue-356 branch August 11, 2023 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] 카테고리에 따른 태그 색상 변화
3 participants