Skip to content
Submitted about 1 year ago

Order summary component [6]

Dean•190
@DeanMustafa
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

I decided to create all challenges to gain some experience. As a self-taught developer, it is almost impossible to gain real-life experience. I think this will help me to get a job in the future.

Community feedback

  • Jeremy Helsel•670
    @JIH7Posted about 1 year ago

    Looks great! One suggestion, and this is something I really need to improve with as well, is to use more semantic HTML. On such a small project there's not a ton of room for this, but the <div> with the class of "card" could totally be an <article> instead. I was going to also suggest making the <h2> an <h1>, but seeing as this is designed to be a small component of a larger website, having an <h2> is a great call actually.

    Marked as helpful
    0
    Dean•190
    @DeanMustafaPosted about 1 year ago

    You are so right. This is actually what I was thinking. @JIH7

    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