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 with mobile first CSS

@Yellow-May

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


Open to Feedbacks

Community feedback

P
David Turnerβ€’ 4,100

@brodiewebdt

Posted

as @meladcodes said every page should have an H1 for accessibility reasons. You shouldn't skip heading levels. You can style them to look however you want. Change the attribution div to a Footer and those things will clear the accessibility warnings.

Download AXE DevTools and you can clear accessibility warnings while you code. https://www.deque.com/axe/devtools/

Hope this helps.

Marked as helpful

1
Meladβ€’ 110

@meladcodes

Posted

Hi, in order to avoid accessibility issues in the future:

  • wrap everything in your body in <main>
  • start your headings with h1 and move up by one level each time

Hope this helps!

0

Anosha Ahmedβ€’ 9,340

@anoshaahmed

Posted

@meladcodes stop copying my comment

0
Meladβ€’ 110

@meladcodes

Posted

@anoshaahmed ?

0
Anosha Ahmedβ€’ 9,340

@anoshaahmed

Posted

@meladcodes you've been copy-pasting the exact same comment i posted

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