@AdrianoEscarabote
Posted
Hi Omar, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
- The links must have an aria-label or sr-only text that tells where the link navigates the user. For example: Visit our Facebook. For images, you should set aria-hidden=”true” to be ignored by screen readers and to avoid redundancy and repetition.
I noticed that once the max-width
is activated, the content starts to go to the side, we can fix it like this:
.wrapper {
margin: 0 auto;
max-width: 1320px;
padding: 0 2rem;
}
The rest is great!
I hope it helps... 👍