@correlucas
Posted
๐พHello @s9trange, congratulations on your first solution!๐ Welcome to the Frontend Mentor Coding Community!
Nice code and nice solution! You did a good job here putting everything together. Iโve some suggestions for you:
1.Add the title of the page inserting in the <head> the tag <title> โ <title>QR Code - Front End Mentor</title>
2.Add the website favicon inserting the svg image inside the <head>
.
<link rel="icon" type="image/x-icon" href="./images/favicon-32x32.png">
3.Add the alt text
to improve accessibility.The alt attribute provides alternative information for an image if a user for some reason cannot view it (because of a slow connection, an error in the src attribute, or if the user uses a screen reader). ---> <img src="./images/image-qr-code.png" alt="QR Code Frontend Mentor">
4.The main heading has the tag <h2>
, in this case, you should replace it with <h1>
since this heading is the main title on this page. Remember that every page should have one <h1>
to declare which is the most important title and that you should follow the hierarchy using the heading sequence (h1, h2, h3, h4, h5)
and never jump a level.
โ๏ธ I hope this helps you and happy coding!
Marked as helpful