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

Davi de Oliveiraโ€ข 50

@QuintaFeiraNoite

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


Feedbacks are always welcome :)

Community feedback

@MelvinAguilar

Posted

Hello there ๐Ÿ‘‹. Good job on completing the challenge !

I have some feedback for you if you want to improve your code.

  • Remove the min-width: 100vw; property from the .card-container selector as this generates a horizontal scrollbar on some devices.
  • Some images do not need to have a descriptive alt attribute, you can leave the alt text of the music icon and background empty. This will hide it from some technologies such as screen readers.

    The main reason for hiding an image from screen readers is that image can be used for decorative elements and may not provide any meaningful information to users of assistive technology. Hiding them from screen readers can prevent confusion and reduce the amount of unnecessary information that is read to users.

    You can read more about this here ๐Ÿ“˜

I hope you find it useful! ๐Ÿ˜„

Happy coding!

Marked as helpful

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