qr-code-component-main

Community feedback

• 

  raya•2,850

  @rayaattaPosted 10 months ago

  Hello 👋Rehab Elashri, congratulations on completing this challenge 🎉.

  I only have two suggestions for you.

  1 The qr code's alt attribute should not just have info that it is a qr code rather it should state where the qr leads I e alt="qr code leading to frontendmentor.io

  2 Since the footer is a sectioning element it should not have text directly written into it but rather it should be wrapped inside a text wrapping element like <a>,<span>,<p> In this case the best element to enclose the text would be a <p>

  I hope this helps.

  Your solution looks good👍

  Happy coding ✌️

  0