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

Order Summary Component | Responsive

Jarrian 150

@godkingjay

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


Please give advice for what I should add/change? Or what/where I should improve...much thanks.

Community feedback

@javascriptooo

Posted

Given the accessibility issues -- you need to include at least the <main> landmark.

The <main> landmark tag can be inserted around your content inside the <body>. Here is some documentation

Adding the <main> landmark will remove all 3 accessibility issues! :)

Hope that helps! Great work!

Steven

Marked as helpful

0

Jarrian 150

@godkingjay

Posted

@javascriptooo I just added it and it solved the 3 issues, thank you so much.

1

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