-
Notifications
You must be signed in to change notification settings - Fork 2
[#9] 버튼 컴포넌트 구현 #33
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
[#9] 버튼 컴포넌트 구현 #33
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
junye0l
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.
양이 많을텐데 버튼 컴포넌트 고생 많으셨습니다!
카카오톡 가이드 : https://developers.kakao.com/docs/latest/ko/kakaologin/design-guide
구글 가이드 : https://developers.google.com/identity/branding-guidelines
소셜 로그인 같은 경우 가이드 라인을 제공해주는데, 한 번 확인 해보시면 좋을 듯 합니다.
코멘트 확인도 부탁드립니다!
src/components/button/Button.tsx
Outdated
| import Icon from "../icon/Icon"; | ||
| import type { IconName } from "../icon/icon-map"; | ||
|
|
||
| interface ButtonProps { |
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.
현재 스토리북 파일에서 disabled props 사용중인데, ButtonProps에 선언이 되어있지않아서 오류가 발생하는 것 같습니다!
wlrnjs
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.
수고하셨습니다!
📄 PR 내용 요약
기본 버튼(기본, 아웃라인)과 아이콘 버튼, 아이콘 + 텍스트 형태의 버튼 컴포넌트를 구현했습니다.
✅ 작업 내용 상세
상세 타입은 스토리북 참고해주세요!
📸 스크린샷 (선택사항)
💬 참고 사항
비활성상태 시안은 없어서 추가로 만들어봤어요.기본 버튼의 기본 너비는100%입니다.