NFT preview card component

Community feedback

• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Good job Melissa

  Keep up the good work

  Your solution looks great however I think that the divider line needs to have less opacity using opacity

  In terms of accessibility issues, page must contain a level one heading, this is caused by a h1 tag not being present in the document.

  Hope this helps

  Cheers

  Marked as helpful

  0
• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Melissa Villegas It looks great! but you should always have one h1 per page of the document.

  <div class="card-text">
            <h1><a href="">Equilibrium #3429</a></h1>
            <p>Our Equilibrium collection promotes balance and calm.</p>
          </div>
  

  Your code is so clean, I hope this is useful to you... Keep coding👍

  Marked as helpful

  0