My second challange here :)

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  Hi RadeQ, good job! I took some time to look at your code and have some ideas for improving it:

  • replace div #wrapper with main tag and wrap the card for improve the Accessibility
  • remove width: 1440px and height: 900px from body
  • set width: 20rem to bgbox id
  • remove width from footer
  • add min-height: 100vh to #wrapper because Flexbox aligns child items to the size of the parent container
  • add a little transition on the element with hover effect

  Overall you did well 😉

  Hope this help and happy coding!

  Marked as helpful

  0