Greenka236 / Tea_cozy

0 stars 0 forks source link

Flexbox #5

Open martoio opened 6 years ago

martoio commented 6 years ago

So, this is supposed to be an exercise on flexbox. You've done a great job at implementing it. One thing I feel I need to share is when to use flexbox and when not to. Not sure if you've touched upon this in your research, so here are some articles that are worth a read:

http://brolik.com/blog/when-to-use-flexbox/ https://jakearchibald.com/2014/dont-use-flexbox-for-page-layout/

Greenka236 commented 6 years ago

Ok, I removed it from the content div, therefore not using it for general page layout, rather centering page with max-width and margin: auto;. But I left all of the other flex elements:

If this is still excessive use of flexbox, please point it out. :)