Qr code component

Community feedback

• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted over 1 year ago

  Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

  ---

  QR iMAGE ALT TEXT 📸:

  • The QR Code Component involves scanning the QR code, the image is not a decoration, so it must have an alt attribute which should explain the purpose of the image.

  ---

  • The alt with Imagem do QR code is not even explaining for what the QR image need to be used.

  ---

  • So update the alt with meaningful text which explains like QR code to frontendmentor.io

  ---

  • Example: <img src="/images/image-qr-code.png" alt="QR code to frontendmentor.io">

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  Marked as helpful

  0
• 

  Jonah Ssegawa•240

  @devjhexPosted over 1 year ago

  Hello @7Wall you've really done an excellent job on this challenge it really show that you push in a lot of effort into it, Great work. As a suggestion, in your HTML instead of using a div for the container class it is recommended to use more accessible elements like the article element which will be so helpful for accessibility most especially screen readers to understand how your code is structured, other than that you have really done it well. Happy coding!!

  Marked as helpful

  0