miagarcia11 / Tea-Cozy

0 stars 0 forks source link

Great effort #1

Open tommygebru opened 5 years ago

tommygebru commented 5 years ago

Great effort on your second submittable project Maria, I can see that your layout resembles the guidelines for Teacozy!

Its clear to me however you need to review nesting and indentation. Also it would really help if you could complete this project per page section. For example work on the header until it is complete and accurate to the guidlines, then ask for feedback and improve it before moving onto the next page section :thumbsup:

tommygebru commented 5 years ago

Overall this project is a little more complex so take some time and work through each section one-at-a-time, as you work through your project. 👀

There are a few minor tweaks you can make to existing elements to clean up your layouts, this includes reviewing certain elements for their margins, paddings or width size that may affect adjacent elements for each page section

One thing that definitely makes it easier is to help create a visual aide and identify the box model for all the webpage elements. For example: *{border: solid 1px orangered ;} or if you prefer *{background: rgba(0,0,0,.1);}

I like to use the rgba values because it shows overlapping HTML really well 👏