NFT Card

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…

  • Adjust the top position in the .main-container .main-img .cover into top: 36.5%;
  • Instead of using section to wrap the whole content, use main tag , refer it with this one Semantics

  Other than that...

  • The project is responsive and utilized the details based on the original design
  • The hover state of the design is done accordingly, specially the hero image
  • The HTML structure is well organized and readable
  • Proper usage of semantic tags
  • The CSS file is all compressed, how come?

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

  Marked as helpful

  0

  ---

  

  Bartek•400

  @imxbartusPosted over 2 years ago

  @RioCantre Thanks for sharing your knowledge, I will use the Sematics that you've mentioned to increase my projects. Css file is compressed, because I have written the styles in sass and I uploaded only the css folder, without the styles.scss.

  0