Skip to content
Submitted over 2 years ago

HTML + CSS

Zdravko•420
@Zdravko93
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Correction of previous 'Order-summary-component' challenge

Community feedback

  • バレンタイン 😈•64,190
    @VCaramesPosted about 2 years ago

    Hey @Zdravko93, some suggestions to improve you code:

    • The “Illustration” and Music Icon serves no other purpose than to be decorative; It adds no value. The Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.

    • Add object-fit: cover to the illustration to have take up the entire container.

    • Remove the height from .payment-description, .hero and .container. It is causing you component to warp.

    • Your mobile view needs some improvement so it looks better on smaller screens.

    Happy Coding! 👻🎃

    0

Please log in to post a comment

Log in with GitHub

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