KeyValueListItem에서 Stack 제거와 Modal export type 문제 해결

[Seolhun]

Summary

• KeyValueListItem에서 KeyItem에 사용되는 Stack을 제거합니다.
• ModalTitleSize가 type으로 export되는 문제를 해결합니다.

Details

As-is

To-be

Stack은 width: 100%, height: 100%을 갖고있는데, 이것은 KeyValueListItem에서 의도되지 않는(불필요한) 스타일입니다. 그러므로, overwrite하지 않고, Stack을 사용하지 않습니다.

Browser Compatibility

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

Windows

• [ ] Chrome - Blink
• [ ] Edge - Blink
• [ ] Firefox - Gecko (Option)

  macOS

• [ ] Chrome - Blink
• [ ] Edge - Blink
• [ ] Safari - WebKit
• [ ] Firefox - Gecko (Option)

References

800

775

[codecov[bot]]

Codecov Report

Merging #817 (aaee879) into next-v1 (5a9e1d4) will decrease coverage by 0.02%. The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           next-v1     #817      +/-   ##
===========================================
- Coverage    67.81%   67.79%   -0.03%     
===========================================
  Files          204      204              
  Lines         2862     2863       +1     
  Branches       784      784              
===========================================
  Hits          1941     1941              
  Misses         804      804              
- Partials       117      118       +1     

Impacted Files	Coverage Δ
...onents/KeyValueListItem/common/KeyItem/KeyItem.tsx	100.00% <ø> (ø)
.../KeyValueListItem/common/KeyItem/KeyItem.styled.ts	100.00% <100.00%> (ø)
.../bezier-react/src/components/Stack/Stack/Stack.tsx	88.88% <0.00%> (-11.12%)	:arrow_down:

---

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 09d8d04...aaee879. Read the comment docs.

[Seolhun]

KeyValueListItem에서 Stack test-coverage 문제가 PR에 영향을 받아야하는걸까요?

[sungik-choi]

KeyValueListItem에서 Stack test-coverage 문제가 PR에 영향을 받아야하는걸까요?

Stack 소스는 전혀 건드리지 않았는데도 테스트 커버리지가 변하네요... 제 생각에도 이건 기대하는 동작이 아닌 거 같아서 이슈로 등록해놓을게요

[Seolhun]

@sungik-choi 머지 강제로 할까요?

[aooen]

@sungik-choi 머지 강제로 할까요?

고고

[ch-builder]

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

The release is available on:

• npm package (@next-v1 dist-tag)
• GitHub release

Your semantic-release bot :package::rocket: