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-Component Solution

Giulia 210

@GiuliaT97

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@Akhil-Nagpal

Posted

Hello Giulia! 👋 Congratulations on completing this project! 🥳

To fix accessibility & HTML errors, you need to fix these things in your code:

  • Always add alternative text to all images you use like: <img src="illustration-1.svg" alt="girl-listening-music">. This specifies an alternate text for an image if the image cannot be displayed.

Other Suggestions 📝

  • Use relative units like em or rem instead of using absolute units like px. This will improve responsiveness. Read this to understand better 👉 Should we use Rem, Em, or Pixel?

  • Background image is not responsive while adjusting the screen size. Use background-size: cover;

  • You missed some properties in the body { } tag. For centering anything use these properties:

display: flex;
text-align: center;
align-items: center;
justify-content: center;

Otherwise! You did a great job! 👏

I hope these suggestions will help you to improve! 😊

Happy Coding!

1

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