@rayaatta
Posted
Hello Parimal Kini๐, congratulations on completing this challenge ๐
I have some suggestions you might find interesting.
1
The Alt attribute should have text including where the qr code leads.
I.e
alt="qr code leading to frontendmentor.io"
2
Remove the <div class= "attribution">
from the main and change it to <footer class="attribution">
And wrap it's text inside a p
element.
This improves accessibility and SEO since it is much more semantic.
3
I noticed font-size: 15px;
in your code.
You should never use pixels when setting font related properties.
Setting font sizes in pixels causes accessibility problems with people who adjusted their default font sizes on their computers.
The best units for setting font size are relative units such as rem and em which can adjust with the user preferences. Checkout this article why font size should never be in pixels
I hope this helps ๐
Your solution is awesome ,I also liked the fact that it is very semantic๐
Happy coding โ๏ธ