Skip to content
Submitted over 2 years ago

Nft preview card component

Luis Angel Rodriguez Frias•40
@LuisRodriguez98
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

Good project, enjoyed it very much. Thanks for the tips.

Community feedback

  • Gabriel_BC•300
    @GabrielBC2021Posted over 2 years ago

    Hello, Just little details

    1.- You are not adding all the hovers, you added the hover to the image, but you are missing adding hover properties to the text Equilibrium #3429 and the text Jules Wyvern.

    2.- You are missing the font weight in some parts of the project, just add them, not big problems

    3.- You are missing the alt attribute in your img tag, just add it (If you don't know what this attribute works for, just tell me and I'll tell you with details).

    Little details that will make your website look better.

    Marked as helpful
    1
    Luis Angel Rodriguez Frias•40
    @LuisRodriguez98Posted over 2 years ago

    @GabrielBC2021 thx, changes made.

    0
  • Hamza-Noah•300
    @Hamza-NoahPosted over 2 years ago

    Increase the width and height of the card until you make it identical with the design To get rid of the accessibility/HTML issues shown in your Report:

    check the report and follow the steps one by one until you have zero accessibility issues. wrap everything in your body in <main> or use semantic tags or give role=" to the direct children of your <body>

    have at least one <h1> in your code Good job!

    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