Skip to content
Submitted almost 4 years ago

Responsive Order Summary Component Challenge using SCSS

Brian Carlo Birondo•320
@Briancarlo24
A solution to the Order summary component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

How was my solution?

Code

Please log in to post a comment

Log in

Community feedback

  • darryncodes•6,350
    @darryncodes

    Hi Brian,

    Great solution, really well done!

    • you could tidy up your accessibility report with a few quick fixes. Change <section> to <main> and remove the id=main. Your .attribution <div> should be a <footer> tag.
    • here is a useful guide

    All the best 🤙

    Marked as helpful
  • Nguyen Duy•245
    @koalalikecode

    I think you should add the hover state of this site

    Marked as helpful
  • P
    Patrick•14,265
    @palgramming

    On your body tag you should change your background size from contain to 100vw 40vh

    Marked as helpful

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