-
Notifications
You must be signed in to change notification settings - Fork 2
[#8] 와인 타입 선택 컴포넌트 제작 #22
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
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.
고생하셨습니다!
코멘트 확인 부탁드립니다!
| htmlFor={name} | ||
| className={cn( | ||
| "py-[7px] pl-2 pr-3", | ||
| "flex items-center justify-center gap-[6px]", |
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.
"flex items-center justify-center" -> "flex-center" 으로 변경해도 좋을 것 같아요!
| errorMsg: string; | ||
| } | ||
|
|
||
| interface TextInputValue extends InputValue { |
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.
interface InputValue {
placeholder: string;
errorMsg: string;
title?: string;
}
두 인터페이스를 합쳐서 title 값만 옵셔널로 지정하는건 어떤가요?
…into design/type-select
…into design/type-select
📄 PR 내용 요약
와인 타입을 선택하는 컴포넌트 입니다.
✅ 작업 내용 상세
와인 타입 선택
hover,checked디자인이 없어 칩 컴포넌트와 동일한 스타일로 디자인 했습니다.사용된 이미지는
/public/image/wine-type디렉토리에 추가했습니다📸 스크린샷 (선택사항)
💬 참고 사항
TextInput컴포넌트는 아직 미완성입니다SelectType컴포넌트만 확인 부탁드립니다!