@edwinc73
Posted
Hey good job completing the challenge! This one is deceptively long due to many smaller interactions to style. Time management takes experience and even then you could go way over time working on a small element.
I would suggest to use something like a wrapper to help you center and define content width.
e.g
HTML
<div class="wrapper"> THE REST OF THE CONENT </div>
CSS
.wrapper{
width: 1200px;
margin: 0 auto;
}
margin 0 auto helps to give margin right and left the maximum value therefore centering the page.
be very careful what types of units you use. e.g 10% padding on the header. I have a large screen so the padding was huge
On the other hand the nav items could use a little more padding around it so less dexterious people can still easily use it
nav ul li{
padding: 10px 5px
}
the navbar should contain links, which should be done using anchor tags this will have the added benefit of allowing for keyboard navigation.
<< EDIT >>
I have included my submission for this challenge so that you could see how i styled it and made structure the sematic html.
