Design comparison
SolutionDesign
Solution retrospective
Thanks Front-end Mentor
Community feedback
- @sulemaan7070Posted almost 2 years ago
hey 😄NZANGMENE HASSAN, congratulations on completing the challenge... here are a few tips to make your site better.
1.Using the picture element for responsively switching between Images is considered as best practice
<picture> <source media="(min-width: 650px)" srcset="img_food.jpg"> <source media="(min-width: 465px)" srcset="img_car.jpg"> <img src="img_girl.jpg"> </picture>
more about picture element here📚
2.Add the functionality🪛 of
closing the mobile menu when user clicks on the overlay
since most users click on an overlay to close the mobile menu...Hope that helps,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