@AdrianoEscarabote
Posted
Hello Subham singh, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:
You have used <br> , using <br> is not only bad practice, it is problematic for people who navigate with the aid of screen reading technology. Screen readers may announce the presence of the element. This can be a confusing and frustrating experience for the person using the screen reader. You can read more in MDN.
Prefer to use max-width or padding and margin!
Images must have alt text unless it is a decorative image, for any decorative image each IMG tag must have empty alt="" and add aria-hidden="true" attributes to make all the assistive technologies of the Web, as screen reader. Learn the differences between decorative/meaningless images vs important content.
To improve the code structure wrap this div:
<div class="attribution">
with the semantic tag footer
The remainder is excellent.
I hope it's useful. 👍
Marked as helpful
