Responsive QR CARD using flexbox

Community feedback

• 

  mubizzy•1,520

  @mubizzyPosted over 2 years ago

  Excellent job on this challenge! your report has a few issues though:

  1. wrap everything in your body in <main> or use semantics

  2. it is a best practice to use both HTML 5 and ARIA landmarks to ensure all content is contained within a navigational region.

  Hope it helps:)...don't forget to mark it as helpful 👍

  0
• 

  Fatlind Shehu•2,230

  @fatlindshehuPosted over 2 years ago

  Hi there, Nice work with the component, even though I would recommend some changes:

  • Center the component using flex;
  • On the body set height: 100vh in order for vertical alignment to work.
  • body { display: flex; justify-content: center; align-items: center;} to center the component.
  • Make sure to remove the margin on the card.

  Keep it up & happy coding...

  0