NFT Preview Card Component Using Tailwind CSS

Community feedback

• 

  Rajaul Islam Ratul•1,250

  @ratul0407Posted 12 months ago

  @ThornenSan congrats on completing this challenge🎉🎉

  You've done a really great job✨. The solution is really great but one thing that caught my eye is that you've used a div tag for almost everything. Whenever you needed a container you used a div. There is nothing wrong with divs they are ideal container. But you should work a little bit on landmarks as they provide a great value for accessibility. You could've used a main as a background container. You could've used a footer for the .attribute div. This will help you to cope with accessibility along the way.

  These are little nick picks and instead of that the solution is great👍

  Have a very nice day and keep up the good work🎉😄😄

  Marked as helpful

  1

  ---

  

  San Thornen•100

  @ThornenSanPosted 12 months ago

  @ratul0407 thank you for the feedback as always

  0