Skip to content
Submitted almost 3 years ago

NFT preview card component

Mehdi Adham•140
@mehdi-adham
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

Solution retrospective

Hey everyone, I've completed the first challenge and I'm looking forward to getting some feedback on this project. Thank you.

Community feedback

  • Rosi•440
    @strosiPosted almost 3 years ago

    Hi Adham! Very good first solution from you. I think the html and css are well structured, just pay attention to the accessibility issues from the report. It says that you need to use sematic HTML to add more meaning to your tags - Semantics in HTML. You could wrap the nft-container in <main> tags and use <h1> for the title element.

    0
    Mehdi Adham•140
    @mehdi-adhamPosted almost 3 years ago

    @strosi Hi Rosi, Thanks so much for the feedback. I'll do that. I'll use semantic HTML in the next solution.

    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