Design comparison
SolutionDesign
Community feedback
- @HassiaiPosted almost 2 years ago
There is no need for the wrapper div , the header logo div and the header-hero-img in the in the html . in the html do this:
<header> <img class="header__img src="" alt = ""> </header> The illustration-image should be in the main , there is no need to give the main a class in the html.Give the text content a div within the main. change the value of the body's background-size to cover, no need for min-heat and font-size to percentage. give the body a width percentage value. for the header giver it a padding-top and bottom value and remover the height. reduce the image of the logo . Give the header a margin bottom for the space between the logo and content. don't give the text-content a font size. Hope am helpful, HAPPY CODING0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord