Skip to content
Submitted about 3 years ago

Mobile Design, HTML, CSS

Fide•20
@fidelp27
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 need a feedback

Community feedback

  • Syazwan Fitri•400
    @syzwnftrPosted about 3 years ago

    Hey, this is good for your first project. I noticed there is an accessibility issues in the solution, which I also had it when I first submit mine. Here's what I'd suggest to get rid of these:

    1. Change the <div> right after <body> to <main>
    2. Change <h2> tag to <h1> tag
    3. Change the <div> before </body> to <section> or you can just put the <div> inside <main> tag.

    I hope this will help.

    Marked as helpful
    0
    Fide•20
    @fidelp27Posted about 3 years ago

    @syzwnftr thanks for your help. Great observations, I'll change my code.

    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