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 using HTML and CSS

@GerryL2

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


Feedback Welcome!

Community feedback

Bryan Li 3,550

@Zy8712

Posted

Your site looks pretty good. Main things you should change are:

  • adding a <main> tag around your page's main content for accessibility purposes
  • adding hover effect for your "change" button
  • centering your container using flexbox instead of what you are using right now. You can do this by assign something like this to your body:
body{
   display: flex;
   flex-direction: column;
   justify-content: center;
   align-items: center;
   height: 100vh;
}

For more one flexbox you could watch these tutorials: link1 link2.

Hope you find this feedback useful 👍

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