NFT preview card component

Community feedback

• 

  T.Praveen•870

  @gwtpraveenPosted over 2 years ago

  Congratulation on completing the challenge.

  • In terms of accessibility issues simply wrap all your content between main tags.

  • Instead of an a tag, I recommend wrapping your img tag in a div tag.

      <div class="image-wrapper">
          <img class="ntf-img" src="./images/image-equilibrium.jpg" alt="">
      </div>
  

  Happy Coding!!

  0

  ---

  

  Khalil Nazari•220

  @khalilnazariPosted over 2 years ago

  Hi @Tim5583,

  Thanks for your guidance. I intentionally wrapped the <img> take with an anchor tag. It's because when it's clicked on the image page should be directed to profile detail. Otherwise <a> is not suggested.

  Cheers!

  0
• 

  AgilitiesZ•310

  @phichayut-pakPosted over 2 years ago

  Congratulations on your fixed submission 🎉🎉 What I would suggest here is :

  1. Make the eye bigger ( the one that appears when you hover the image )
  2. change the font and font size of the title Equilibrium #3429
  3. Optional Do some transitions to make the website more alive

  Happy Coding!

  0

  ---

  

  Khalil Nazari•220

  @khalilnazariPosted over 2 years ago

  Hi @phichayut-pak,

  Thanks for the highlight. Sure the font sizing is not accurate because I have not registered for the pro version.

  Cheers!

  0