Vanza Setia• 27,795
@vanzasetia
Posted
Hello, Anton! 👋
Nice work on this challenge! 👍
Two things can be done better.
- It's great that the alternative text of the QR code tells the users the exact use case of the QR code which is going to navigate the users to the Frontend Mentor site. But, I would recommend removing the word "Scan". It is not a call to action like a link or button so telling the users about the image is good enough.
- Put the attribution outside the
main
landmark. It should live on thefooter
landmark. It's simply because the attribution is a different type of content from the card.
Hope this helps. 🙂
2