Skip to content
Submitted about 3 years ago

Order Summary Component with Flexbox and Grid - First Challenge

Milen Nenkov•80
@milen-nenkov
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

Hello all, This is my first ever project outside of a tutorial. I would appreciate any pointers, advice or full blown constructive criticism you might have for a budding newbie web dev. Thank you and until we meet again. Best Regards Milen

Community feedback

  • Haobijam Chetan Singh•195
    @chetanhaobijamPosted about 3 years ago

    Great job, @milen-nenkov. I would suggest to increase the font-size of the Heading "Order Summary" a bit to match the design. And use html5 semantic elements such as header, main, section, article, footer, etc. instead of usual divs to fix the html accessibility issues in your website. Keep on coding... Chetan.

    Marked as helpful
    1
    Milen Nenkov•80
    @milen-nenkovPosted about 3 years ago

    Thank you for the feedback @chetanhaobijam. I updated the code to factor in the accessibility issues you pointed out.

    0
  • Robert K•80
    @RobertK0Posted about 3 years ago

    I'm no pro myself, but I think the grid and flex box were well used. The only thing I would advise is to try playing out with centering the "wrapper" vertically on the page, instead of going around that by using a fixed margin-top. Other than that, well done!

    1
    Milen Nenkov•80
    @milen-nenkovPosted about 3 years ago

    Thanks Robert :)

    1

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