NFT preview card component solution

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  Hello there! Awesome work with this challenge. Looking at your solution, I think you should know the following as well…

  • Instead of div, alternatively wrap the whole content with semantic tags, for HTML structures, refer it with this one Semantics
  • Import the attribution style in CSS file or remove the style tag
  • Clean the whitespaces in the code
  • Include description with the alt in image tags but it's optional

  In the brighter side...

  • Good job in the hover state of the design, specially the hero image
  • The project is responsive and used the details based on the original design
  • The CSS file is well organized with proper usage of properties and sizing elements
  • The HTML structure is organized and readable

  Above all, the project is well implemented. Keep up the good work!

  Marked as helpful

  0

  ---

  

  Samuel Pletain•390

  @samuelpletainPosted over 2 years ago

  @RioCantre Thank you for the feedback!

  0