Skip to content
Submitted about 3 years ago

Summary card

Santhosh•590
@santhoshpsd2
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

Any Feedback would be appreciated

Community feedback

  • Gaurav Singh•100
    @gauravnumberPosted about 3 years ago

    Try to use Semantic HTML like header, main, section. Replace <div class="container"> with <main>

    Marked as helpful
    0
  • Gaurav Singh•100
    @gauravnumberPosted about 3 years ago

    replace <h3> Order Summary</h3> with <h1> Order Summary</h1> It gives accessibility issue.

    First heading always start with h1 not h3.

    Order matter h1, h2, h3, h4, h5 Don't use h3 then h5. Use like this h3 and h4. One heading incremental is consider good.

    Marked as helpful
    0

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