design-system icon indicating copy to clipboard operation
design-system copied to clipboard

Prepare for DSv5

Open craigpalermo opened this issue 2 years ago • 1 comments

Changes included:

  • Upgrade to Styled System v5
  • Upgrade to Styled Components v5
  • Replace boxShadows in theme with new shadows values
  • Replace old borderRadii in theme with new values
  • Remove fullWidth prop from Button
  • Remove fallback to Styled System for non-palette colors from Box and all components that extend it (e.g. no more hex values or CSS colors allowed)

Breaking changes:

  • Unit tests may fail if ResizeObserver is not defined

craigpalermo avatar Aug 24 '22 14:08 craigpalermo

Codecov Report

Base: 95.72% // Head: 95.53% // Decreases project coverage by -0.18% :warning:

Coverage data is based on head (cf449b7) compared to base (88a50bd). Patch coverage: 91.76% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1193      +/-   ##
==========================================
- Coverage   95.72%   95.53%   -0.19%     
==========================================
  Files         177      177              
  Lines        2407     2418      +11     
  Branches      569      573       +4     
==========================================
+ Hits         2304     2310       +6     
- Misses         49       50       +1     
- Partials       54       58       +4     
Impacted Files Coverage Δ
packages/autocomplete/src/Autocomplete.js 100.00% <ø> (ø)
...ges/carousel/src/ArrowButton/ArrowButton.styles.js 100.00% <ø> (ø)
packages/core/src/Checkbox/Checkbox.tsx 100.00% <ø> (ø)
packages/core/src/FormField/FormField.tsx 91.30% <ø> (ø)
packages/icons/src/Svg.js 0.00% <0.00%> (ø)
packages/menu/src/MenuList/MenuList.js 100.00% <ø> (ø)
packages/modal/src/ModalHeader.js 100.00% <ø> (ø)
packages/modal/src/SmallModalHeader.js 100.00% <ø> (ø)
packages/popover/src/Popover/Popover.js 100.00% <ø> (ø)
...kages/popover/src/PopoverContent/PopoverContent.js 100.00% <ø> (ø)
... and 50 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Aug 24 '22 16:08 codecov[bot]

This pull request introduces 2 alerts when merging c00fbc606569111308384ff35cb0ede0186e56e6 into ffe64b5bfec3dcebe789d0dd7afa04341f29dc29 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

lgtm-com[bot] avatar Oct 18 '22 16:10 lgtm-com[bot]

This pull request introduces 2 alerts when merging 83de8b81c40e415bb09c543d2efc71ddb07fabfc into ffe64b5bfec3dcebe789d0dd7afa04341f29dc29 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

lgtm-com[bot] avatar Oct 18 '22 16:10 lgtm-com[bot]

This pull request introduces 1 alert when merging 47dda1dbb42a4e266bef9b627775010aa323e45e into ffe64b5bfec3dcebe789d0dd7afa04341f29dc29 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Oct 18 '22 16:10 lgtm-com[bot]

This pull request introduces 1 alert when merging cbb56103095ba26e321dad3ebfb782d9dc6b58ce into ffe64b5bfec3dcebe789d0dd7afa04341f29dc29 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Oct 18 '22 16:10 lgtm-com[bot]

This pull request introduces 1 alert when merging 8dbf9d37a2445abed7ae231d6460c5af367ecd80 into ffe64b5bfec3dcebe789d0dd7afa04341f29dc29 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Oct 18 '22 16:10 lgtm-com[bot]

This pull request introduces 1 alert when merging 84d175de9b0f98f4e89bc859d9e5e561616f11f7 into ffe64b5bfec3dcebe789d0dd7afa04341f29dc29 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Oct 18 '22 16:10 lgtm-com[bot]

This pull request introduces 1 alert when merging 199e7c7ac7cff602565e979740e01e7971560e0a into f53e03e2a7247666a1b9d1ca303c62cc3df779da - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Oct 19 '22 17:10 lgtm-com[bot]

This pull request introduces 1 alert when merging 6dfd33843ea7e9886012f8a7ad04ca4352d0c00a into f53e03e2a7247666a1b9d1ca303c62cc3df779da - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

lgtm-com[bot] avatar Oct 19 '22 17:10 lgtm-com[bot]