@AdrianoEscarabote
Posted
Hi Danilo, how are you? Welcome to the front-end mentor community! I really liked the result of your project, but I have some tips that I think you will enjoy:
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.
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.
The rest is great!
I hope it helps... 👍
Marked as helpful
@OneBitMance
Posted
@AdrianoEscarabote That is so amazing, i didn't even have the slightest idea of the 'aria-hidden' and empty 'alt' property and how they contribute to the web, let alone <br> tags resembling a problem for screen readers. Thank you so much for this!