Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Order-Summary ( HTML , CSS )

@AbdallahGO

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I hope I make it right after the very useful feedback I get 😊

Community feedback

@VCarames

Posted

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 GitHub
Discord logo

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