Frontend Mentor - QR code component solution

Community feedback

• 

  Usama Ahmed•220

  @UsamaBinKashifPosted over 2 years ago

  Thank you mate. <3

  0
• 

  Eduardo•20

  @eduardovalientePosted over 2 years ago

  Hola Usama! El resultado final quedó muy bien! Solo note en el HTML que no usaste la etiqueta MAIN, en lugar de eso, utilizaste un div con la clase "main", el resto impecable!

  Felicitaciones por el trabajo!! 👏👏

  Saludos amigo!!

  0
• 

  Travolgi 🍕•31,400

  @denieldenPosted over 2 years ago

  Hi Usama, I took some time to look at your solution and you did a great job!

  Also add main tag and wrap the card for Accessibility and remove all unnecessary code, the less you write the better as well as being clearer: for example the div with box1 class is superfluous

  Overall you did well :)

  Hope this help and happy coding!

  0

  ---

  

  Usama Ahmed•220

  @UsamaBinKashifPosted over 2 years ago

  @denielden Thank you so much for your feedback mate. I'll try to do better in the next challenge. ❤️

  1