QR code component using HTML, Sass and Flexbox

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…

  • Include description with the alt in image tags. Like this line <img src="./images/image-qr-code.png" alt="" /> but it's optional
  • This line <div class="text"> could be wrap with section tag

  On the brighter side..

  • Well organized HTML structure
  • Proper usage of semantic tags
  • Code is readable and neat
  • The design is well implemented
  • Good use of width sizes from the main content to the image
  • The project is responsive
  • Good use of Flexbox for the alignment

  Above all, The design looks good. Keep it up!

  0

  ---

  

  Nuel•70

  @nuelobetoPosted over 2 years ago

  @RioCantre thank you for your feedback and corrections. I appreciate you!!

  0