-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: CheckBox 구현 #9
Conversation
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.
고생하셨습니다!! 열일한나... ( •̀ ω •́ )✧👍 궁금한 점 코멘트로 남겨두겠습니당
@nijuy 엇 이 부분 다시 보시면 CheckBox 직접 클릭했을 때 isSelected 제어 안되는 거 확인하실 수 있어요! 여기 부분은 State가 실시간으로 반영되지 않아서 이렇게 보이는 것 같은데 다른 방법이 있을지 모르겠네요 ㅠㅡㅠ... |
앟. 직접 클릭했을 때는 상태가 잘 변하는데 controls 탭에서 저번에 알려주셨던 예전 배포판(?) 보니까 checkbox는 실시간 제어가 되는데 한나 말대로 toggle은 안되네요 멋쓱 ! |
@nijuy 그 예전 배포판에서 Checkbox가 isSelected로 제어가 되는 이유가 CheckBox 자체를 클릭할 수 있는 기능을 지원하지 않아서 그런 걸거에요! 클릭이 안되더라구욥 큐ㅠㅠㅠ |
input:disabled + .label { | ||
${({ $size }) => | ||
$size === 'small' && | ||
css` | ||
${typos.button4} | ||
`} | ||
} |
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.
@HyunsDev 👍 div (icon-padding)로 체크와 라벨 사이 여백 넣는 것보다 gap으로 주는 게 깔끔해보여서!! 최고ㅇㅔ요
|
1️⃣ 어떤 작업을 했나요? (Summary)
2️⃣ 알아두시면 좋아요!
handleWrapperClick 쪽에서 isDisabled일 경우 return, 아닐 경우 checked 값에 따라 isSelected 변경되도록 구현해두었습니다. 혹시 더 좋은 방법이 있다면 알려주시면 감사하겠숩니당
3️⃣ 추후 작업
PlainButton 이어서 작업할게용