-
Notifications
You must be signed in to change notification settings - Fork 0
[Feat] Button 컴포넌트 디자인 구현 #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
The head ref may contain hidden characters: "4-feat/Button-\uCEF4\uD3EC\uB10C\uD2B8-\uC81C\uC791"
Conversation
PR 검증 결과✅ TypeScript: 통과 |
PR 검증 결과✅ TypeScript: 통과 |
KyeongJooni
left a comment
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.
코멘트 확인해주세요! 수고하셨습니다! :)
| 'gap-[--spacing-xxs]', | ||
| 'rounded-[--radius-l]', | ||
| 'w-[163px] h-[44px]', | ||
| 'px-[var(--spacing-xl)] py-[var(--padding-m)]', |
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.
패딩과 같은 것들은 스페이싱에 사용되는 토큰 값을 사용하는 것보다 추가로 padding 전용 토큰 값을 만들어서 사용하는게 더 좋을 것 같습니다! 😄
PR 검증 결과✅ TypeScript: 통과 |
PR 검증 결과✅ TypeScript: 통과 |
KyeongJooni
left a comment
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.
LGTM 👍
|
확인 감사합니다. 머지 진행하겠습니다! |
✨ 주요 변경사항
공통 Button 컴포넌트에 피그마 스타일을 적용해 디자인 토큰(index.css)으로 기반해 버튼 스타일을 만들었습니다.
📝 작업 상세 내용
✅ 체크리스트
Close #번호추가📸 스크린샷 (선택)
🔍 기타 참고사항
🔗 관련 이슈