Skip to content
Submitted almost 4 years ago

I used flexbox and used buttons (not links)

David Chavez•190
@codeDavidC
A solution to the Order summary component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

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

Code

Please log in to post a comment

Log in

Community feedback

  • Neil Khatri•620
    @nkhatri7

    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

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