channel-io / bezier-react

React components library that implements Bezier design system.
https://main--62bead1508281287d3c94d25.chromatic.com
Apache License 2.0
184 stars 46 forks source link

Tag, Badge 컴포넌트의 ClassName생성을 줄이는 작업 #566

Closed quino0627 closed 2 years ago

quino0627 commented 2 years ago

Description

as-is to-be
스크린샷 2021-07-14 오후 5 55 51 스크린샷 2021-07-14 오후 5 42 18
다른 색상에 대해 다른 className을 generate 다른 색상도 같은 className을 generate

Browser Compatibility

OS / Engine 호환성을 반드시 확인해주세요.

Windows

(Option) Issues

(연관된 PR이나 Task / 특정 시점까지 머지하면 안됨 / 번역 추가 필요 / ...)

codecov[bot] commented 2 years ago

Codecov Report

Merging #566 (15c4695) into next-v1 (832e676) will increase coverage by 5.47%. The diff coverage is 57.42%.

Impacted file tree graph

@@             Coverage Diff             @@
##           next-v1     #566      +/-   ##
===========================================
+ Coverage    43.13%   48.61%   +5.47%     
===========================================
  Files          143      143              
  Lines         1975     1981       +6     
  Branches       404      401       -3     
===========================================
+ Hits           852      963     +111     
+ Misses         788      694      -94     
+ Partials       335      324      -11     
Impacted Files Coverage Δ
src/components/Avatars/Avatar/Avatar.tsx 53.84% <ø> (ø)
...mponents/Avatars/AvatarGroup/AvatarGroup.styled.ts 0.00% <0.00%> (ø)
src/components/Avatars/AvatarGroup/AvatarGroup.tsx 0.00% <ø> (ø)
.../Avatars/CheckableAvatar/CheckableAvatar.styled.ts 0.00% <0.00%> (ø)
...onents/Avatars/CheckableAvatar/CheckableAvatar.tsx 0.00% <0.00%> (ø)
src/components/Header/Header.styled.ts 0.00% <ø> (ø)
src/components/Header/Header.tsx 0.00% <ø> (ø)
src/components/ListItem/ListItem.tsx 66.66% <ø> (ø)
src/components/TagBadge/Badge/Badge.tsx 0.00% <ø> (ø)
src/components/TagBadge/Tag/Tag.tsx 0.00% <0.00%> (ø)
... and 44 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 02d05d9...15c4695. Read the comment docs.

jwoo0122 commented 2 years ago

아직 작업 중이신거죠?

quino0627 commented 2 years ago

아직 작업 중이신거죠?

@jwoo0122 다 되었습니다!

ch-builder commented 2 years ago

:tada: This PR is included in version 1.0.0-next-v1.29 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

quino0627 commented 2 years ago

useFoundation hook 은 날려주세요.

styled-omponents에도 기본적으로 제공하는 hook이라 둬도 괜찮을 것 같다고 생각했는데 다른 의견이 있으실까요? @jwoo0122

jwoo0122 commented 2 years ago

useFoundation hook 은 날려주세요.

styled-omponents에도 기본적으로 제공하는 hook이라 둬도 괜찮을 것 같다고 생각했는데 다른 의견이 있으실까요? @jwoo0122

사용처가 없는 hook 이라 굳이 선제적으로 제공할 필요는 없다고 생각해서 드린 피드백이었습니다. 정확히는 이 PR 에서 styled.attrs 를 사용하기로 바꾸면서 지워졌어야 하는 hook 으로 이해했어요.

quino0627 commented 2 years ago

아하 필요해질 때 export 하자는 방향이신 거군요 말씀주신 대로 본 pr에서는 필요없어지게 된 hook이 맞구요, 저는 어떤 방향이던지 상관 없다고 생각하기 때문에 따로 revert pr올리겠습니다