Skip to content
Submitted about 3 years ago

Responsive order summary card

maia•300
@maiaflow
A solution to the Order summary component challenge
Preview (opens in new tab)Code (opens in new tab)
Code

Please log in to post a comment

Log in

Community feedback

  • Eric Salvi•1,350
    @ericsalvi

    Hey Maia,

    Great job with your first submission!!! I like how you structured your layout on the submission. The class names make sense and the different sections that make up this component are easy to understand. Also thank you for the call-out in your repository!

    A few things I would suggest either fixing now or remembering to keep an eye out for future submissions is to fix any accessibility or HTML issues.

    https://www.frontendmentor.io/solutions/responsive-order-summary-card-LpTS2rYbNT/report

    Something I like to do also is to make sure to remove any HTML commented-out sections or unnecessary CSS.

    The last thing I noticed is that your background image does not span the entire body width. You could easily add background-size: 100% to the body selector to fix this.

    Cannot wait to see future submissions by you!

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