
Submitted over 2 years ago
Huddle Landing Page Section
#bem#progressive-enhancement#semantic-ui#accessibility
@FreivysDev
Design comparison
SolutionDesign
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Hassiai
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 CODING
Join 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