Order summary using flexbox

Community feedback

• 

  Yui•50

  @yteraiPosted over 2 years ago

  Your code looks neat and super clean! I'm gonna try that next time :)

  1
• 

  maia•300

  @maiaflowPosted over 2 years ago

  Great work on this Toyin! It looks perfect! In your code, I like your semantic class names, I'm going to work on using those next time. Something else I want to try next time that we both didn't do is organizing your css into sections with commented out headers. Another one of those is deleting any commented out lines as a final clean up before submitting. Excited to see your next one!

  1
• 

  Eric Salvi•1,350

  @ericsalviPosted over 2 years ago

  Hey Toyin,

  Great job on your first submission. Checked out your code and it looks clean!

  A few things I would try to do for your next challenges is to always check out the validation for accessibility or HTML. Either do this before you submit by using axe DevTools extension or you can check out the report that FrontendMentor generates here.

  One thing I would suggest doing before you submit is to take all of your CSS and run it through a tool such as https://autoprefixer.github.io/. What it will do is generate any required additional CSS to make this submission work on all platforms and browsers. The best way to almost be fully compatible.

  For your links instead of using # for these types of components, I'd suggest using javascript:void(0) instead as this will not refresh the page when a link is clicked, unlike using #.

  Great work again and I cannot wait to see more of your work.

  Keep up that momentum!

  1