NFT preview card component

Community feedback

• 

  ABDULRAZZAK ALSSSOUKI•180

  @AbdalrzakalsoukiPosted almost 2 years ago

  Hello, that is look great

  here some suggestions:

  • Accessibility:

  1. Heading should only increase by one, means you go from h3 to h4, not h3 to h5, this is important for SEO

  • HTML validations:

  1. use heading with article

  • General:

  1. importing the fonts in CSS is not really good for performance, unless you need a lot of customization, better to include it in HTML head
  2. this is optional but you can use some variables with CSS, it make modifying much easier
  3. do not put the custom CSS before Google fonts, this will reduce the performance
  4. you can minimize the code for better performance
  5. use a meta description

  hope this is helpful, all luck

  Marked as helpful

  2

  ---

  

  Gabriela•180

  @PaulaR-05Posted almost 2 years ago

  @Abdalrzakalsouki awesome tips, thanks a lot :)

  1

  

  ABDULRAZZAK ALSSSOUKI•180

  @AbdalrzakalsoukiPosted almost 2 years ago

  @PaulaR-05 you are welcome

  1
• 

  Sameer Singh•560

  @SameerJS6Posted almost 2 years ago

  "Nice one, buddy, but you forgot to add hover effects on the link tags (Creators Name, Price) and on the main image. You also forgot to add a cursor: pointer to some links. Always make sure to decrease the heading tag only by one. For example, if you have used h1, you should not use h4 directly on the page; it should only decrease one by one."

  Thank You,

  Marked as helpful

  1

  ---

  

  Gabriela•180

  @PaulaR-05Posted almost 2 years ago

  @SameerJS6 Thank you :) I'm going to correct everything you said, thanks a lot!

  0