@andreasremdt
Posted
Hey @ricardosilvasx,
Congrats on solving the challenge, well done! Your solution looks good, I like that you wrapped everything in a main
tag which is a good practice. Here are some additional suggestions:
- Try and incorporate a heading. Every website needs at least one heading to explain the content. Always start with the highest level (
h1
) and then descend down when it makes sense. In this example, I would make the text Improve your front-end skills by building projects the main heading. - You don't need the
div.qrcode
, just apply the styles of it to themain
element directly. - Your image doesn't have any
width
andheight
attributes, which I would advice you to set. They allow the browser to better calculate the image dimensions and hence optimize rendering, which results in a better loading experience.
Keep up the good work and let me know if you have any questions!
Marked as helpful