refactor: JDS Checkbox, Radio 타입 안정성 개선#502
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
WalkthroughCheckbox와 Radio 컴포넌트의 타입 정의를 ChangesCheckbox/Radio 타입 안정성 개선 및 스타일 변경
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
|
||
| export const checkboxLabel = style({ | ||
| whiteSpace: "nowrap", | ||
| zIndex: 10, |
There was a problem hiding this comment.
📝 overlay()의 절대배치 ::before/::after가 라벨 텍스트를 덮는 걸 방지하기 위한 스태킹 보정
💡 작업 내용
typeprop 외부 오버라이드 차단*.types.ts내부interface/type혼용을type으로 통일💡 자세한 설명
외부에서
input의type을 전달하면type="checkbox"/type="radio"가 덮어써져 해당type셀렉터에 의존하는 스타일까지 깨질 수 있어,type을 prop으로 받지 않도록 막았습니다. 아울러 두 컴포넌트의 타입 정의에서interface와type이 혼용되어 있던 부분도type으로 통일했습니다.작업 중
Checkbox.Label의z-index가 누락된 부분을 발견해 함께 반영했습니다.📗 참고 자료 (선택)
📢 리뷰 요구 사항 (선택)
✅ 셀프 체크리스트
closes #501
Summary by CodeRabbit