Hi, I hope this feedback is helpful. I'll write as a list and hopefully you can work through each item as I'm off the rest of this week. Others are available for more help on Discord if you need it - just start a new post in the #Help channel :)
footer
is a distinct landmark frommain
. It should sit after it- The body will need to be made into a flex column if this is the approach you're using to center the component on the screen
- Give main some padding so the component can't touch screen edges (Alternatively, you could give the card some margin on all sides, or give the body a little padding)
- The card must have a max-width in rem
- The alt text on the image needs to be a proper description. Remember, alt is not code, it is important content! For example "QR code to FrontendMentor.io" would be a good description for this image
- You are loading either the incorrect fonts or the incorrect widths. Check these in the styleguide and amend what you are downloading and applying
- Try to get the font sizes and appearance much closer to the design. That includes family, weight, size, line height and vertical margins
- ALWAYS include a modern CSS reset at the very start of the styles. That will do lots of important things, but one of them is set images to display block max-width 100%
p.s. Don't forget to update the attribution link so it goes somewhere meaningful! The link for your name could go to your frontend mentor profile page, for example
Marked as helpful
0