segmentio / evergreen

🌲 Evergreen React UI Framework by Segment
https://evergreen.segment.com
MIT License
12.39k stars 832 forks source link

remove css-nil classes from glamor #1280

Closed mshwery closed 3 years ago

mshwery commented 3 years ago

Overview This PR removes those pesky css-nil classnames. This occurs whenever there are no styles to give to glamor:

{}
{ foo: false }
null
undefined

We are typically only dealing with {} in our style config hook, but this will uniformly omit the css-nil class.

Screenshots (if applicable) NA

Documentation

netlify[bot] commented 3 years ago

✔️ Deploy Preview for evergreen-storybook ready!

🔨 Explore the source changes: 9b1ad0542faa29e3460d3011918699fedbb7a475

🔍 Inspect the deploy log: https://app.netlify.com/sites/evergreen-storybook/deploys/61016f7eee83410008990f30

😎 Browse the preview: https://deploy-preview-1280--evergreen-storybook.netlify.app