refactor: (JDS) Listbox를 표면 없이 사용할 수 있는 옵션 추가 - #711
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (5)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. Walkthrough
ChangesListbox 표면 제어
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature · Severity of issue fixed: Low 🚥 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. 토끼가 Listbox 곁을 깡충 뛰네 Comment |
| label?: string; | ||
| width?: SelectDimension; | ||
| height?: SelectDimension; | ||
| surface?: boolean; |
There was a problem hiding this comment.
Select가 surface prop을 받아 Listbox로 그대로 넘기도록 구현해야 할지 고민하다, 현재의 방식을 채택했는데요. 이유는 다음과 같아요:
- 컨테이너 내부에 listbox를 포함하는 요구가 현재 DatePicker 하나뿐이에요.
- Select는 현재 배럴에 공개되어 있어, 그대로 prop을 열어 버리면 공개 API가 불필요하게 늘어난다 생각했어요.
- 표면 제거(유/무)는 Select가 아닌 Listbox의 책임이라고 봤어요.
🖼️ VRT baseline 재생성 완료
|
🎨 스토리북 알파 배포 완료📦 배포 주소
ℹ️ 정보 |
💡 작업 내용
surfaceprop 추가selectContainer를 recipe로 전환해 표면 관련 선언만 variant로 분리💡 자세한 설명
작업 진행 이유
Listbox의 컨테이너는 기본적으로 공중에 떠 있는 드롭다운 카드로 그려지고 있습니다. 하지만, 이미 표면이 있는 컨테이너 내부에 리스트를 넣는 경우에는 카드가 이중으로 겹치고, 좌우 패딩 때문에 옵션 폭도 좁아져 표면 없이 사용할 수 있는 옵션을 추가했습니다. (DatePicker 연/월 선택)
표면만 variant로 분리
레이아웃은 surface 각 모드가 공유하고 있어
base에 두고, 표면에 해당하는 선언만surfacevariant로 옮겼습니다.스토리
Listbox는 배럴에 공개되지 않아 스토리가 없던 컴포넌트였지만, 이번에 표면 prop을 추가하면서useSingleSelectState+useListbox조합 방법을 문서로 추가 작성했습니다.✅ 셀프 체크리스트
closes #710
Summary by CodeRabbit