Responsive Landing Page using HTML and CSS
Design comparison
Solution retrospective
Eagerly, looking forward to receiving some feedback on my project.
Community feedback
- @HassiaiPosted almost 2 years ago
Add aria-label attribute
aria-label=" "
to the <a> to fix the error with the code, the value for the aria label is the description of the link. Add alt attributealt = " "
to the img tag <img> to also fix the error issue with the page, the value of the alt is the description of the image.Wrap <img class="main-icon" src="./logo.svg"> in a header tag. Give the main, the header and the footer, a max-width of 1440px, width of 80% and margin: 0 auto;
Add background-size of cover to the body. Give .main-img a max-width of 100% , there is no need to give it display: inline-block. In the media query give .main-box a width of 100%. a Give main a margin-top and bottom values, instead of giving it to .main-img and .main-box
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful2
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