NFT preview card using CSS flexbox

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  Hello there! Good job in completing this challenge. Regarding your solution, I would like you to notice the following…

  • Clean the whitespaces in the code
  • Instead of div, alternatively wrap the whole content with semantic tags, for HTML structures, refer it with this one Semantics
  • Remove height: 75vh; in the div.container ruleset
  • To understand more about the hero image hover state, refer it with this example Overlay in Image
  • The project is not responsive, use Flexbox for the aligment or margin:auto or better adjust the properties in the media queries

  In the brighter side...

  • The hover state of the design is done properly
  • The CSS files is well structured and proper usage of sizing elements

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

  0

  ---

  

  Teju Adedoyin•230

  @t3ju-doyinPosted over 2 years ago

  @RioCantre Thank you soo much, will look into them

  0