Closed Dogdriip closed 1 year ago
Latest commit: 486e8ed1a9cc52d4dedcbc03268dd927d658a661
The changes in this PR will be included in the next version bump.
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
Merging #873 (486e8ed) into next-v1 (46caba3) will increase coverage by
0.13%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## next-v1 #873 +/- ##
===========================================
+ Coverage 70.87% 71.00% +0.13%
===========================================
Files 204 204
Lines 2884 2894 +10
Branches 797 807 +10
===========================================
+ Hits 2044 2055 +11
+ Misses 722 721 -1
Partials 118 118
Impacted Files | Coverage Δ | |
---|---|---|
...ier-react/src/components/Divider/Divider.styled.ts | 100.00% <100.00%> (ø) |
|
...es/bezier-react/src/components/Divider/Divider.tsx | 100.00% <100.00%> (+16.66%) |
:arrow_up: |
...ages/bezier-react/src/foundation/Rounding/index.ts | 100.00% <100.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
- 윈도우 포함, 브라우저별로 잘 표시되는 지 확인부탁드려요!
- 사방 마진을 제거하는
withoutIndent
prop이 추가되면 사용처에서 더 쉽게 사용할 수 있을 거 같습니다. 아마 대부분withoutSideIndent
,withoutIndent
둘만 사용하게 될 거 같아요
withoutIndent
를 하나 더 추가하자는 말씀이 맞을까요?
- 윈도우 포함, 브라우저별로 잘 표시되는 지 확인부탁드려요!
- 사방 마진을 제거하는
withoutIndent
prop이 추가되면 사용처에서 더 쉽게 사용할 수 있을 거 같습니다. 아마 대부분withoutSideIndent
,withoutIndent
둘만 사용하게 될 거 같아요
- 윈도우는 집 가서 체크해보겠습니당 🏡
- 지금 props에서
withoutIndent
를 하나 더 추가하자는 말씀이 맞을까요?
네 추가하는 방향이 맞습니다! 테스트용 윈도우 노트북이 사내에 있으니, 크로마틱 배포된 스토리북으로 접속하셔서 확인하셔도 될거에요
Browser Compatibility만 체크되면 Approve 하겠습니다!
Browser Compatibility만 체크되면 Approve 하겠습니다!
최대한 체크해 보았는데 대부분의 브라우저에서 문제 없는 것 같습니다. PR 본문에 체크해두었으니 확인 부탁드립니다! 🙇
Summary
Divider
컴포넌트에withoutParallelIndent
prop을 추가합니다.Divider
컴포넌트에 Radix를 적용합니다.Details
변경점
Divider
가 더 이상HTMLHRElement
가 아닌HTMLDivElement
입니다.& > hr
등과 같이 태그 이름을 지정해 스타일링한 경우가 있다면 마이그레이션이 필요합니다.withoutParallelIndent?: boolean = false
prop을 추가합니다.withoutSideIndent
와 네이밍을 최대한 맞췄습니다.withoutIndent?: boolean = false
prop을 추가합니다.decorative: boolean | undefined
prop을 추가합니다.round1
을 추가합니다.Radix 도입과 관한 이야기
Root
,Track
,Range
,Thumb
를 각각 제공합니다.bezier-react
의styled
등을 사용합니다.@radix-ui/react-polymorphic
을 이용한 Polymorphic component를 extend하여 bezier를 주입합니다.@radix-ui/react-polymorphic
는 Deprecated되었으며, 아래 방법이 훨씬 간단하기 때문에 굳이 이렇게까지 할 필요는 없다고 생각했습니다.asChild
prop을 이용해 bezier 컴포넌트에 accessibility를 주입합니다.Divider.tsx
의 일부입니다.Browser Compatibility
OS / Engine 호환성을 반드시 확인해주세요.
Windows
macOS
References
seperator
: https://www.w3.org/TR/wai-aria-1.2/#separator