QR Component with Flexbox

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  Hi HutHut, great job!

  Remove all margin from qr-code-component because with flex they are superfluous, for center the card add align-items: center to body. Read here -> flex guide

  Overall you did well! Hope this help ;)

  Marked as helpful

  0

  ---

  

  HutHut•310

  @j-hutchisonPosted almost 3 years ago

  @denielden cheers deniel! I knew there was something didn't feel quite right about that margin 😅 will take a look later!

  Thank you for reviewing for me!

  1

  

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  @j-hutchison You are welcome! And don't forget to mark my comment as helpful :) thanks

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! HutHut Good Job on challenge

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body <main class="container"></main>

  -> Always use h1 first and then h2, h3 and so on

  -> Learn more on accessibility issues

  If this comment helps you then pls mark it as helpful!

  Have a good day and keep coding 👍!

  0