annarw / prj-rev-bwfs-tea-cozy

0 stars 0 forks source link

Summary #5

Open josephgupkim opened 5 years ago

josephgupkim commented 5 years ago

Score: 16/16

Your Tea Cozy project looks great (on both the code side and web side)!

++ Consistent indentations and whitespace. ++ Proper flexbox HTML structure. ++ Shows understanding in positioning with flexbox. ++ Site matches design spec in terms of content and styling. ++ Working navigation bar.

-- Could have a more efficient method in making the gap disappear. -- Content of site not centered on wider screens.

Keep up the awesome work! 👍

annarw commented 5 years ago

Thanks for the feedback Joseph :)

Added the margin of 0px to the body and deleted the max-width of 1430px I had added to nav. This did the trick, it looks a lot better now (on wider screens) and makes more sense.

image