
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Hassiai
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 helpful
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