@Hassiai
Posted
Wrap <div class="logo"> and <div class="container"> within the main tag and <div class="icons"> in the footer tag to fix the accessibility issues. click here for more on web-accessibility and semantic html
wrap the icons in the ul tag and li instead of separate divs
There is no need to give the body a width and height value. Change the value of the background-size to cover.
For the left and right sides space of the page, give the main and the footer a max-width of 1440px, a width of 80% and margin:0 auto.
main, footer{
max-width:1440px;
width:80%;
margin: 0 auto;
}
To center the icons change the value of the justify-content to center.
Use relative units like rem or em as unit for the padding, margin, width values and preferably rem for the font-size values, instead of using px which is an absolute unit. For more on CSS units Click here
Hope am helpful.
Well done for completing this challenge. HAPPY CODING