Responsive QR code component

Community feedback

• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • Always Use Semantic HTML instead of div like <main> <header> <footer> , etc for more info

  1

  ---

  

  Kamrul Hassan•100

  @khshakibPosted almost 3 years ago

  @Old1337 thank you for your feedback. i will definitly work for that

  1
• 

  optimusprime202•1,200

  @optimusprime202Posted almost 3 years ago

  Hey @khshakib, Congratulations, you got it right!

  0

  ---

  

  Kamrul Hassan•100

  @khshakibPosted almost 3 years ago

  @optimusprime202 thank you dear

  0
• 

  Sridhar•10

  @SridharDneshPosted almost 3 years ago

  Hello 👋, Font size could have been reduced a little to match with the challenge provided.

  0

  ---

  

  Kamrul Hassan•100

  @khshakibPosted almost 3 years ago

  @SridharDnesh thank you for your suggetion

  0
• 

  Jérémy•1,020

  @jrmydixPosted almost 3 years ago

  Hi! Well done!

  If this can help, I have a suggestion:

  • Fix accessibility issues: to do so, change your <div class="container"> to <main>.

  If you have any questions feel free to ask, I'll try my best to help and answer! 🙂

  0

  ---

  

  Kamrul Hassan•100

  @khshakibPosted almost 3 years ago

  @jrmydix thank you for favor

  1