Skip to content
Submitted over 3 years ago

Order summary card

Mohamed Hussein•305
@MohamedHussein05
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

Community feedback

  • Juveria Dalvi•1,375
    @Juveria-DalviPosted about 3 years ago

    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
    Mohamed Hussein•305
    @MohamedHussein05Posted about 3 years ago

    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-DalviPosted about 3 years ago

    @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

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