-
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
[Refactor] Switch 컴포넌트에 토큰 적용 #34
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.
전 좋습니다 🙌 토큰이 수정돼서 styled-system 에서도 변경이 생기지 않나유?
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.
저도 좋아요!!
생각해 보니.. 저도 r
값을 토큰을 이용하도록 수정해야겠네요..!
넵 wow-ui styled-system에 반영해뒀고 왜인지는 모르겠지만 루트에 styled-system 폴더가 있길래 같이 삭제했어요! |
🎉 변경 사항
Switch 컴포넌트에 토큰 적용했습니다.
🚩 관련 이슈
🙏 여기는 꼭 봐주세요!
theme에 정의되어 있는 radius랑 spacing은 기존에 정의된대로 사용하려면 아래처럼 써야 하더라구요
radius를 반복하지 않고 쓰는 게 더 좋을 거 같기도 하고 radius, spacing을 제외한 다른 토큰들이 그렇게 되어 있기도 해서 아래와 같이 쓸 수 있도록 radius, spacing 객체 구조를 변경했습니다.