Skip to content
Submitted almost 3 years ago

NFT Preview Card

rfsan•20
@rfsan
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

  • Lty•110
    @LithyiePosted almost 3 years ago

    Hi, you might use <a> instead of <p> for the "equilibrium" and the creator name as its highly suggested to be links.

    Also you should put your code inside <main> </main> after <body> and change one if your <p> to <h1> cuz every page should have one <h1> (its mostly part of the good practice for SEO and accessibility things) to clean your accessibility issues

    For CSS, i'm not fan of putting active state separated to its related content, it can be confusing imo, thats all i can say, i'm more used to code in LESS now so its automaticly converted to css, i havent read pure css for long time 😅

    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