Skip to content
Submitted almost 3 years ago

Responsive landing page using CSS Flexbox

Jorge Guedes•30
@jorgeguedess
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...NFT preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Community feedback

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

    To get rid of the accessibility/HTML issues shown in your Report wrap everything in your body in <main> OR give role="" to the direct children of your <body> ... Click here to read more

    Good job! :)

    Marked as helpful
    1
  • Aakash Verma•6,710
    @skyv26Posted almost 3 years ago

    Hi!

    To solve accessibility issues:

    1. Either Change your code

    FROM

    <div class="attribution">
    Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. 
    Coded by <a href="https://github.com/Jorge-Guedes" target="_blank">Jorge Guedes</a>.
  </div>

    TO

    <footer class="attribution">
    Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank" aria-label="challenged by">Frontend Mentor</a>. 
    Coded by <a href="https://github.com/Jorge-Guedes" target="_blank" aria-label="developer-name">Jorge Guedes</a>.
  </footer>

    OR you can simply add role="footer" attribute in your existing code. but don't forget to add aria-label attribute as I added above.

    Happy New Year ;)

    Marked as helpful
    1

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