[Feat] FavoriteButton 컴포넌트 제작#40
Hidden character warning
Conversation
PR 검증 결과✅ TypeScript: 통과 |
PR 검증 결과✅ TypeScript: 통과 |
KyeongJooni
left a comment
KyeongJooni
left a comment
There was a problem hiding this comment.
고생하셨습니다! 코멘트 확인 부탁드립니다!! :)
| export const FavoriteButton = ({ | ||
| defaultActive = false, | ||
| onToggle, | ||
| ariaLabel = '찜', | ||
| }: { | ||
| defaultActive?: boolean; | ||
| onToggle?: (isActive: boolean) => void; | ||
| ariaLabel?: string; | ||
| }) => { |
There was a problem hiding this comment.
props 타입을 다른 컴포넌트에서 감쌀 때 재사용을 위해 interface로 별도로 분리해서 export 하는게 더 좋을 것 같다고 생각하는게 어떻게 생각하시나요...?!
There was a problem hiding this comment.
좋은 의견 감사합니다! 재사용을 고려해 props를 interface로 분리하고 export 했습니다 👍
There was a problem hiding this comment.
variant 속성이 없는 경우 tva 의존성을 주입할 필요가 없다고 저는 개인적으로 생각해서 Styles.ts로 분리했을 것 같습니다! 제 개인적인 생각이니 참고는 해보시는 것도 좋을 것 같아요!
There was a problem hiding this comment.
의견 감사합니다! 피그마를 다시 살펴 보니 수리점 내역 화면에서도 테두리 색상이 반전된 FavoriteButton이 사용되고 있어서, default/inverse variant를 추가하면서 tv를 유지했습니다 또한 추후 카드 컴포넌트에 FavoriteButton이 추가될 경우 사이즈가 달라질 가능성도 있다고 생각해, 확장성을 고려하여 현재 구조를 유지했습니다! 필요하다면 styles로 분리하는 방향으로 수정해 보겠습니다 😀
PR 검증 결과✅ TypeScript: 통과 |
PR 검증 결과✅ TypeScript: 통과 |
2 participants
✨ 주요 변경사항
📝 작업 상세 내용
✅ 체크리스트
Close #번호추가📸 스크린샷 (선택)
FavoriteButtonDemo.mov
🔍 기타 참고사항
🔗 관련 이슈