Skip to content
Submitted almost 3 years ago

Frontend Mentor - Order summary card

LeeFuuChang•90
@LeeFuuChang
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

  • Anosha Ahmed•9,300
    @anoshaahmedPosted almost 3 years ago

    To get rid of the accessibility issues shown in your Report:

    • wrap everything in your body in <main> ... OR use semantic tags ... OR give role="" to the direct children of your <body> ... Click here to read more
    • have at least one <h1> in your code
    • start your headings with <h1>, and move up by one level each time

    Great job! :)

    Marked as helpful
    0
    LeeFuuChang•90
    @LeeFuuChangPosted almost 3 years ago

    @anoshaahmed I’ve fixed the issues that you mention. But, the accessibility issue kept poping up, it says that the <a> tag must have text inside? if I didn’t misunderstand it.

    0
    Anosha Ahmed•9,300
    @anoshaahmedPosted almost 3 years ago

    @LeeFuuChang did it say "discernable text"? if so, that means u need to add a aria-label="" Click here to learn more

    Marked as helpful
    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