NFT preview card component

Community feedback

• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • Always Use Semantic HTML instead of div like <main> <header> , etc for more info

  if my solution has helped you do not forget to mark this as helpful!

  Marked as helpful

  1

  ---

  

  Sam Mathew•80

  @dayumsamPosted over 2 years ago

  @Old1337 Okay, I will keep this in mind the next time, In this particular project where will you suggest I use the <main> tag>.

  Thank you so much for your feedback.

  1
• 

  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 also take notes of the following…

  • Import the attribution style and font style in CSS file and remove the style tag. Advisable not to use two style tags within the head
  • Wrap the whole content with specific tag like main. For HTML structures, refer it with this one Semantics
  • Include description with the alt in image tags. Like this line <img src="./images/icon-view.svg" alt="">
  • Add the inline style in the CSS file as well.

  Above all, The design looks good. Keep it up! Hope this is helpful!

  0
• 

  Hexer•3,660

  @EmmanuelHexerPosted over 2 years ago

  Great work overall man. Keep it up

  0

  ---

  

  Sam Mathew•80

  @dayumsamPosted over 2 years ago

  @Phalcin Thank you!!

  0