Skip to content

Conversation

@yongb2n
Copy link
Contributor

@yongb2n yongb2n commented Nov 17, 2024

📌 PR 템플릿

🏷️ PR 타입 (PR Type)

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

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

📝 요약 (Summary)

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

  • 우리의 소중한 라벨 컴포넌트입니다 (요긴하게 사용 될 예정)

🔍 상세 내용 (Describe your changes)

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

  • required prop에 Highlight 컴포넌트 적용 -> (@KingNono1030 감사합니다 🤩)
  • 스토리북 구현
  • RadioInput 타입 에러 해결
  • layout prop으로 스타일 분기처리 작업

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

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


✅ 체크리스트 (Checklist)

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

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

📸 스크린샷 (선택 사항)

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

2024-11-17.9.48.16.mov

📝 기타 사항

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

@yongb2n yongb2n self-assigned this Nov 17, 2024
@yongb2n yongb2n linked an issue Nov 17, 2024 that may be closed by this pull request
@yongb2n yongb2n marked this pull request as draft November 17, 2024 12:09
@yongb2n yongb2n requested review from KingNono1030 and yellowjang and removed request for KingNono1030 and yellowjang November 17, 2024 12:09
@KingNono1030
Copy link
Contributor

라벨에서 required 일 때, * 하이라이트 하는건 text/highlight 컴폰너트 사용하시면 될거 같아요 !

@yongb2n yongb2n marked this pull request as ready for review November 17, 2024 12:50
@yongb2n yongb2n changed the title [#47] ✨ 공통 컴포넌트 Label 생성 [#47] ✨ 공통 컴포넌트 Label 구현 Nov 17, 2024
@@ -0,0 +1,29 @@
import clsx from 'clsx'

import { Highlight } from '../text'
Copy link
Contributor

Choose a reason for hiding this comment

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

절대 경로로 바꿔 주세요 !
@/components/common/text

Copy link
Contributor Author

Choose a reason for hiding this comment

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

변경 완료!

@KingNono1030
Copy link
Contributor

피그마 마이 페이지에서 유저 정보 수정 폼을 보면, 라벨과 인풋이 flex row 로 되어 있는 듯 한데, 혹시 여기 스타일도 고려해서 분기처리 추가해주실 수 있나요 ?

@KingNono1030
Copy link
Contributor

스토리까지 꼼꼼하게 구현해주셨네요, 감사합니다 !

@yongb2n
Copy link
Contributor Author

yongb2n commented Nov 17, 2024

피그마 마이 페이지에서 유저 정보 수정 폼을 보면, 라벨과 인풋이 flex row 로 되어 있는 듯 한데, 혹시 여기 스타일도 고려해서 분기처리 추가해주실 수 있나요 ?

앗 이건 제가 못 봤네요 ㅋㅋㅋ 작업하겠습니다

@yongb2n yongb2n merged commit d9684b9 into dev Nov 18, 2024
1 check passed
@yongb2n yongb2n deleted the feat/common-component-label branch November 18, 2024 07:44
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] [공통 컴포넌트] Label

3 participants