Skip to content
Submitted about 3 years ago

I used flexbox and used buttons (not links)

David Chavez•190
@codeDavidC
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

Are the naming conventions, best-practice, or syntax correct? Please let me know. Thank you!

Community feedback

  • Neil Khatri•620
    @nkhatri7Posted about 3 years ago

    Hey David,

    The layout and your code look great! The only thing I'd suggest is using Semantic HTML. So for example instead of writing <div class="main-content"> it would be better to use <main>. This blog post by Vincent Bidaux has a good explanation of what HTML5 semantic elements are and why they are beneficial.

    But other than that, great job!

    Marked as helpful
    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