NFT preview card component

Community feedback

• 

  Fer•3,970

  @fernandolapazPosted over 1 year ago

  Hi 👋, just a couple of suggestions:

  • When there is a hover state over an element it means that it is interactive, so there must be an interactive element around it (like a link or a button). So, we should use a <a> or <button> to wrap the image (depending on what happened when clicking on it). Also 'Equilibrium #3429' and 'Jules Wyvern' should be wrapped with the <a> tag.

  • The icons are decorative images and therefore need an empty alt attribute to be ignored by a screen reader.

  I hope it’s useful 🙂

  Great solution.

  Regards,

  Marked as helpful

  0

  ---

  

  yas•390

  @yosrajalaliPosted over 1 year ago

  Hi👋 About interactive elements I figured it out after submitting my solution by checking others' solutions(lazy enough to change it right know🥱). but I didn't know about empty alts. I appreciate your help so much🙏. @fernandolapaz

  1