NFT preview card component

Community feedback

• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hello Shane Pinder ! Congo 👏 on completing this challenge

  Just one thing:

  • Consider using h2-h6 elements to add identifying headings to all sections.

  happy Coding😀

  Marked as helpful

  0

  ---

  

  P

  Shane Pinder•190

  @ShanePinderDevPosted over 2 years ago

  @Crazimonk Thanks for the feedback!

  1
• 

  Muh Suryadi Triputra Rahman•400

  @msuryaditriputraRPosted over 2 years ago

  Hi There 👋

  I wanna congrats you for success to finish this challenge 😎🎉

  but I have some feedback for your codes ⚒

  1. I think you don't need to use sections tag, use div instead.
  2. Use box-shadow for adding shadow on your card, not border.

  I hope this helps and can improve your code 🙌

  Happy Coding ☕ Danbo

  Marked as helpful

  0

  ---

  

  P

  Shane Pinder•190

  @ShanePinderDevPosted over 2 years ago

  @msuryaditriputraR Thanks very much for your feedback!

  0
• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Hey ShanePinderDev, good job for completing this challenge. Keep up the good work.

  Your solution looks great, however the height of the card is supposed to be a bit bigger to match the design.

  In terms of the validation errors simply wrap all your content within <main> tags to get rid of validation errors.

  I hope this helps

  Cheers, Happy coding 👍

  0

  ---

  

  P

  Shane Pinder•190

  @ShanePinderDevPosted over 2 years ago

  @TheCoderGuru Thanks for the feedback!

  0