Design comparison
Community feedback
- @HassiaiPosted almost 2 years ago
Add the alt attribute to the <img> to fix the error issue. Replace <h2> with <h1> to fix the accessibility issue.
Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on this watch this https://youtu.be/N5wpD9Ov_To there is no need to give the h1 a font-size. Give the img a max-width of 100% instead of a fixed width value. There is no need to give the img a height value. there is no need for .container and its styling in the css. Use the colors that were given in the styleguide.md in the challenge folder you downloaded.
Fo r a more responsive width, use max-width instead of width in .main-section , there is no need for the height value
Hope am helpful. HAPPY CODING
Marked as helpful0
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