Skip to content
Submitted 11 months ago

NFT preview card

Elena Gonzalez•70
@ElenaGonzalez2000
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

Hello! Do you have any advice on good practices? 🙌

Community feedback

  • Daniel 🛸•44,310
    @danielmrz-devPosted 11 months ago

    Hello @ElenaGonzalez2000!

    Your solution looks great!

    I have a suggestion for improvement:

    • For semantic reasons, and since your project does not have a main title, you can replace the <a> with <h1> and add hover effect to make it look like a link when hovered over. Or at least wrap the text with <h1> inside the <a> tag.

    Other than that, you did an excellent job!

    I hope it helps!

    Marked as helpful
    0
    Elena Gonzalez•70
    @ElenaGonzalez2000Posted 11 months ago

    @danielmrz-dev Thank you very much for the advice, I will take it into account! ☺

    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