NFT preview card component layout [With SASS/SCSS]

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…

  • This line <div class="attribution"> can be wrap with footer tag or place it inside the article tag
  • The Sass file can be break into smaller parts, for instance, a separate file for variables.
  • Include description with the alt in image tags but that is optional

  Other than that...

  • The project is responsive and utilized the details based on the original design
  • Well done in using Sass for styling, the composition and properties are done accordingly
  • Proper usage of semantic tags
  • The HTML structure is well organized and readable
  • The hover state of the design is done properly, specially the hero image

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

  Marked as helpful

  1

  ---

  

  Iván De León Lino•600

  @Lino-OTMPosted over 2 years ago

  @RioCantre I really appreciate these comments more than you can believe, I will implement them in my next projects to improve more and more and have very good practices!

  I really appreciate your time! Thank you very much!

  0