Vanza Setia• 27,795
@vanzasetia
Posted
Greetings, Fran! 👋
Great work on this challenge! Your solution looks pretty good! 😀
I notice that there's a media query in the stylesheet which can be removed. It's only repeating the max-width property which has been declared at line number 22.
The image content is the most important content of the site. So, I highly suggest adding some alternative text to it. Alternative text like, "QR code for frontendmentor.io" would be good.
Lastly, I wouldn't recommend limitting the height of the body element, it will not allow the users to scroll the page if the page content needs more height. Use min-height instead.
Hope this helps.
Marked as helpful
2
