Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Mobile Design, HTML, CSS

Fide 20

@fidelp27

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I need a feedback

Community feedback

@syzwnftr

Posted

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

@fidelp27

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

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