Skip to content
Submitted about 3 years ago

order summary

Jiao Liu•20
@Jiao-Liu
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

any suggestion will be much appreciated!

Community feedback

  • Erel Ropeta•385
    @ereljapcoPosted about 3 years ago

    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-LiuPosted about 3 years ago

    @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

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