NFT preview card component

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  Hey Parthib45, congratulations on completing the challenge! You did a great job 😉

  Let me give you some little tips for optimizing your code:

  • use h1 for the title of card and not a simple p
  • remove all margin from main tag
  • use flexbox to the body to center the card. Read here -> best flex guide
  • after, add min-height: 100vh and margin: 0 to body because Flexbox aligns child items to the size of the parent container
  • add transition on the element with hover effect
  • instead of using px use relative units of measurement like rem -> read here

  Hope this help! Happy coding 😁

  0
• 

  Divine Obeten•2,415

  @Deevyn9Posted over 2 years ago

  Hi, try centering the div in the body on the desktop view. The nft price on the mobile needs adjustment too. Happy coding

  0