flexbox

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 2 years ago

  Hello John, You have done a good work! 😁

  Some little tips to improve your code:

  • you can use article tag instead of a simple div to the container card for improve the Accessibility
  • use img element instead of a styled div for the qr image
  • remove all unnecessary code, the less you write the better as well as being clearer: for example the div without scope
  • use min-height: 100vh to section tag instead of height, otherwise the content is cut off when the browser height is less than the content

  Keep learning how to code with your amazing solutions to challenges.

  Hope this help 😉 and Happy coding!

  Marked as helpful

  0