Design comparison
Solution retrospective
Hello community this is my solution. Your feedback would be appreciated π
Community feedback
- @correlucasPosted about 2 years ago
πΎHello Andrea, congratulations for your new solution!
You've done a really good work here putting everything together creating the css structure with BEM. Your product card its perfect, there's not much to say about it, make it better and concise, you can manage the product image inserting the tag
<picture>
to wrap both desktop and mobile images together in the same tag, and render each image depending of the device by the settings for the width you'll insert in the html.Here's a guide for this tag:
https://www.w3schools.com/tags/tag_picture.asp
If you want to do something more, you can check the
lighthouse
on devtools to check the page performance and see what the report say to improve something if you can.π I hope this helps you and happy coding!
Marked as helpful0@AndreaDev14Posted about 2 years agoHello @correlucas, Thank you for your feedback. π
0
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