Qr code component main

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…

  • Instead of div, alternatively wrap the whole content with semantic tags, for HTML structures, refer it with this one Semantics
  • Include description with the alt in image tags but it's optional

  In the brighter side...

  • The HTML structure is organized and readable
  • The CSS file is done properly with proper composition and sizing elements
  • The project is responsive and utilized the details based on the original design

  Above all, The project is awesome! Keep it up!

  1

  ---

  

  Saul•1,260

  @xsaulPosted over 2 years ago

  @RioCantre Thank you so much for your comments, I'll consider them! And about the alt in the image, I didn't know what to write so I just left it empty. 😅

  0