@AdrianoEscarabote
Posted
Hi codecolastudio, 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.
To improve the structure of the html document, prefer to wrap all the content with the main
tag, since this challenge is just a component, it needs a main
tag to know what the main content of the page is!
To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is and follow the sequence h1-h5
<h1>Improve your front-end skills by building projects</h1>
The rest is great!
I hope it helps... 👍
Marked as helpful
@cruz-jerwin15
Posted
@AdrianoEscarabote Thanks man, your review helps me a lot.