Skip to content
Submitted over 3 years ago

NFT preview card component

Ayoub Amkhazzou•280
@ayoam
A solution to the NFT preview card component challenge
Preview (opens in new tab)Code (opens in new tab)
Code

Please log in to post a comment

Log in

Community feedback

  • PhoenixDev22•16,830
    @PhoenixDev22

    Greeting Ayoam , Excellent solution, really well done!

    To tackle the accessibilty issues , I would suggest to:

    • 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

    • Page should contain a level-one heading, So you can replace <h2> by <h1> and an anchor tag in <h2 class="header-title">Equilibrium #3429</h2>.

    • Add alternate text to the images. Setting the alt text for the images in <div class="header-footer">to an empty string. That's because the purpose of those images are mostly decorative and need not be announced by assistive technology like screen readers.

    I really hope this feedback helps. Happy coding

    Marked as helpful
  • Ayoub Amkhazzou•280
    @ayoam

    thank you mate , i appreciate your feedback :)

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