@AdrianoEscarabote
Posted
Hi fsuropaty, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
To remove the scrollbar, we can do this:
body {
margin: 0;
}
The most appropriate element for this content:
<p class="content-title">Improve your front-end skills by building projects</p>
since this is the main title and every page needs to have an h1
tag to inform what the main title is!
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.
The rest is great!
I hope it helps... 👍
Marked as helpful
@isaacfleurivil
Posted
@AdrianoEscarabote Thanks a ton Adriano, the link you sent me was extremely helpful.