NFT preview using flexbox and CSS Grid

Community feedback

• 

  Travolgi 🍕•31,300

  @denielden

  Posted over 3 years ago

  Hi Thobo, great work on this challenge! 😉

  Here are a few tips for improve your code:

  • You can add the effect :hover creating a div that appears on hover. I used tailwind but you can still see and understand which css properties you can use to do the same. Look here -> my solution
  • use main tag for wrap the card or content of the page, not the content of card
  • remove all margin from .container class because with grid they are superfluous
  • after, add min-height: 100vh and margin: 0 to body because Grid aligns child items to the size of the parent container
  • instead of using px use relative units of measurement like rem -> read here

  Overall you did well 😁 Hope this help!

  00

  Marked as helpful