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

Jiao Liu 20

@Jiao-Liu

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


any suggestion will be much appreciated!

Community feedback

@ereljapco

Posted

Hello! It looks good. But I think you can maximize the use of the <body> and <main> tags. On your CSS, you can transfer the properties of the .main-container to the body tag and the properties of the .wrapper to the .main-container. Then add the class .main-container to the <main> tag and change the <header> tags into <divs>.

And for the CSS, you can still add styles to the <button> tags.

Good job! :)

0

Jiao Liu 20

@Jiao-Liu

Posted

@erelita thank you so much for the feedback! your helps light my way on coding!

0

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