Design comparison
Solution retrospective
I hope I make it right after the very useful feedback I get 😊
Community feedback
- @VCaramesPosted about 2 years ago
Congrats @AbdallahGO on completing this challenge!
I noticed that for the background you placed it in your HTML code. Unfortunately, this isn't best practice. When it comes to decorative images, its better to use the CSS code to implement them by using the background-image property.
I attached a link to you a YouTube video explaining how use the property properly:
https://www.youtube.com/watch?v=zHZRFwWQt2w
Since this a product card, to make your content semantically correct, you'll want to wrap your content in an <article> element.
Heres a link with more details on it:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/article
Happy Coding!
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