Skip to content

Conversation

KingNono1030
Copy link
Contributor

@KingNono1030 KingNono1030 commented Nov 8, 2024

📌 PR 템플릿

🏷️ PR 타입 (PR Type)

아래 해당 사항에 체크해 주세요.

  • 🐛 버그 수정 (Bugfix)
  • ✨ 기능 개발 (Feature)
  • 🎨 코드 스타일 변경 (Code style update) - 포매팅, 로컬 변수 등
  • ♻️ 리팩토링 (Refactoring) - 기능 변화 없음, API 변경 없음
  • 🛠️ 빌드 관련 변경 (Build related changes)
  • 📝 문서 내용 변경 (Documentation)
  • 🔄 기타 (Other) - 설명 작성

📝 요약 (Summary)

PR의 목적과 간단한 설명을 적어주세요.

공통 컴포넌트인 칩 개발


🔍 상세 내용 (Describe your changes)

변경 사항을 구체적으로 작성해 주세요.

  • Chip 개발 (테스트, 스토리북)
  • DeletableChip 개발 (테스트, 스토리북)
  • tailwind config 수정
    • lineHeight 미적용 발견, 피그마 스타일 letter spacing 미적용 발견
  • mock 파일 생성 (jest 가 svg 를 파싱하지 못하는데, jest 가 svg 가 포함된 컴포넌트를 만나면 svg 가 아닌 더미로 처리하도록)

🔗 관련 이슈 또는 링크 (Issue Number or Link)

이슈 번호나 관련 링크가 있다면 추가해 주세요.


✅ 체크리스트 (Checklist)

PR 작성 시 아래 사항들을 점검해 주세요.

  • 빌드가 성공적으로 되었나요?
  • 코드에 주석을 추가했나요?
  • 모든 테스트가 통과했나요?
  • 관련 문서가 업데이트되었나요?

📸 스크린샷 (선택 사항)

변경 사항이 UI와 관련이 있다면 스크린샷을 추가해 주세요.


2024-11-08.15.12.54.mov

image

📝 기타 사항

PR과 관련된 기타 사항이 있다면 적어주세요.

chip 은 라벨에 따라 다른 스타일을 가지도록 했습니다.

@KingNono1030 KingNono1030 self-assigned this Nov 8, 2024
@KingNono1030 KingNono1030 linked an issue Nov 8, 2024 that may be closed by this pull request
3 tasks
@KingNono1030 KingNono1030 requested a review from yongb2n November 8, 2024 07:40
Copy link
Contributor

@yongb2n yongb2n 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 +87 to +98
heading1: ['40px', { lineHeight: '54px', letterSpacing: '-0.02em' }],
heading2: ['28px', { lineHeight: '36px', letterSpacing: '-0.02em' }],
heading3: ['24px', { lineHeight: '32px', letterSpacing: '-0.02em' }],
heading4: ['22px', { lineHeight: '30px', letterSpacing: '-0.02em' }],
heading5: ['20px', { lineHeight: '28px', letterSpacing: '-0.02em' }],
title1: ['18px', { lineHeight: '26px', letterSpacing: '-0.02em' }],
title2: ['16px', { lineHeight: '24px', letterSpacing: '-0.02em' }],
body1: ['16px', { lineHeight: '24px', letterSpacing: '-0.02em' }],
body2: ['15px', { lineHeight: '22px', letterSpacing: '-0.02em' }],
body3: ['14px', { lineHeight: '20px', letterSpacing: '-0.02em' }],
caption1: ['13px', { lineHeight: '18px', letterSpacing: '-0.02em' }],
caption2: ['12px', { lineHeight: '16px', letterSpacing: '-0.02em' }],
Copy link
Contributor

Choose a reason for hiding this comment

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

아앗..!!!

@KingNono1030 KingNono1030 merged commit f712fbb into dev Nov 8, 2024
2 of 3 checks passed
@KingNono1030 KingNono1030 deleted the feat/common-component-chip branch November 8, 2024 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] [공통 컴포넌트] Chip
2 participants