Responsive NFT preview card using flexbox

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

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

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

  • add main tag and wrap the card for improve the Accessibility
  • to make it look as close to the design as possible use flex to .r class for center the eye icon and remove from it top and right properties
  • remove all margin from .main-section class
  • use flexbox to the body to center the card. Read here -> best flex guide
  • after, add min-height: 100vh to body because Flexbox aligns child items to the size of the parent container
  • add transition on the element with hover effect

  Hope this help! Happy coding 😁

  Marked as helpful

  1

  ---

  

  why-not-phoenix•90

  @why-not-phoenixPosted over 2 years ago

  @denielden Thank you so much for the feedback, I'll get to work on these tips right away.

  1