Design comparison
Community feedback
- @malek-btPosted almost 2 years ago
Hey !, how are you ? I really liked the result of your project, but I have some tips that I think you will enjoy:
1.The <img> alt attribute is used to specify the alternate text for an image. It is useful when the image is not displayed. It is used to give alternative information for an image.So make sure to make it not empty.
2.Replace <div class="background-img"> width <section class="background-img"> for semantic page . You can read more about semantic here: HTML Semantic Elements.
3.Remove the commented code in your html file.
4.I see your README file is MAL structured. README is a very important aspect of making projects, especially if you want other people to see it. You should read the README file and the README-template file to make nice one.
The rest is good, I hope those tips will help you! 👍
Good job, and happy coding! 😁
1
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