NFT preview card component

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  Hello there! Nice job in completing this challenge. Regarding your solution, I would like to take notes of the following…

  • Instead of div, alternatively wrap the whole content with main tag . Use semantic tags, for HTML structures, refer it with this one Semantics
  • Include description with the alt in image tags but that's optional
  • Add the hover state of the design. For the hero image , refer it with this one Overlay in image. For the fonts, add color: hsl(178, 100%, 50%); and cursor: pointer; in .text1 and span
  • Try viewing the compositions in the CSS file, one section needs to be fix
  • Add border: 1px solid white; and border-radius: 50%; in the #avatar ruleset
  • Increase the font size of .text1 into font-size: 22px;
  • Add padding: 1rem 0; in the .textss ruleset

  In the brighter side...

  • The project is responsive and utilized the details based on the original design
  • The CSS file have proper properties and sizing elements
  • The HTML structure is well organized and readable

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

  Marked as helpful

  0