Use margin-top or padding-top for vertical spacing

masakudamatsu / line-height-picker

A front-end web app that helps web designers/developers pick the best line-height value for their websites

https://line-height-picker.vercel.app

MIT License

2 stars 1 forks source link

Closed masakudamatsu closed 4 years ago

masakudamatsu commented 4 years ago

For the possibility of using the adjacent sibling combinator to change the vertical spacing depending on what comes above.

For consistency

It can be either, but given the collapsing of margin-top / margin-bottom, we should use either of them only throughout the code.

masakudamatsu commented 4 years ago

No longer relevant as we use spacer div for vertical spacing (#186 )