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 card

@MohamedHussein05

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Juveria Dalvi 1,375

@Juveria-Dalvi

Posted

Hey..😊 So the accessibility issue says you should have <main> tag which is a semantic tag, wrap your all divs inside the main tag except for the div with class attribution that should be wrapped in <footer> tag.

And below is the solution of error in accessibility - Error: <a href="/" class="logo logo-img-1x">

Fix it with : <a href="#" class="logo logo-img-1x">

Also <html lang="en"> this is the right way to declare html tag

1

@MohamedHussein05

Posted

Thank you for your feedback @juuveria, but this report has something wrong with it .

I deployed the website again if you checked it you will see that every thing it say I already have it from the beginning 😅

0
Juveria Dalvi 1,375

@Juveria-Dalvi

Posted

@MohamedHussein05 ok then you should go and view' report and generate new report then the issue will get solved

Your website shows 404 error on preview check if you linked your netlify link the right way also update your solution in view report if you made changes

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