search

wix-incubator / mjml-react

React component library to generate the HTML emails on the fly
MIT License
993 stars 50 forks source link

Make MjmlSection children optional #21

Closed chriskalmar closed 5 years ago

chriskalmar commented 5 years ago

For MjmlSection I believe the children prop should be optional as well, as one could use MjmlSection purely for decorational purposes.

e.g. using it to display a border

<MjmlSection
  background-color="lightcoral"
  padding="3px 0 0 0"
  border-radius="2px 2px 0px 0px"
/>
mastertheblaster commented 5 years ago

Thanks for PR, and i think it makes sense

chriskalmar commented 5 years ago

thanks so much for the fast review 🙏