csnx icon indicating copy to clipboard operation
csnx copied to clipboard

Use common `Space` type across components

Open jamesmockett opened this issue 9 months ago • 3 comments

What are you changing?

  • Use the common Space type in components that accept spacing values as props.
  • Fixes a bug where spacing styles are not applied if space.0 is used due to 0 being falsy.
  • Update Stack stories to better show how spacing is applied.
  • Expands Inline and Stack stories to include examples of all spacing values.
Screenshot 2024-05-08 at 15 00 27

Why?

  • It removes a small amount of duplication and avoids having to update spacing values in multiple places. New spacing values were added last year, but some components still lack support for these as their types were not updated at the same time.

jamesmockett avatar May 08 '24 13:05 jamesmockett