Frontend-Mentor-nft-preview-card-component

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…

  • 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 and remove the style tag
  • Add border-radius: 10px; and adjust the height into height: 98%; in the .eye ruleset
  • Add cursor: pointer for the hover state of the design

  Other than that...

  • The CSS file is well organized and proper usage of sizing elements
  • Well done in using root variables in styling
  • The HTML structure is well organized
  • The project is responsive and utilized the details based on the original design
  • Good job in implementing the hover state of the design, specially the hero image

  Above all, The project is awesome! Keep it up!

  0