@AdrianoEscarabote
Posted
Hi Andrea, 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.
To improve the design of the project, we could use a max-width
to prevent the content from stretching at higher resolutions, and for the background images not to break together we can use a div for just the content and another just for it.
so in the div that the content will be, we would put the max-width
and the margin
to center. In the image div we would let it stretch, or if you decide to use a background-image
we would put a background-repeat: repeat-x;
!
The rest is great!
I hope it helps... 👍
Marked as helpful
@MirMurr
Posted
@AdrianoEscarabote
Hi Adriano, I am good, I hope you are doing well!
Thank you very much for your help! Especially with the aria-labels, it has solved many errors.
I used the max-width properties in this project (I have done some corrections) and added more media-queries to improve responsiveness. Despite of the corrections, everything still seems to be too wide on the screenshot. When I open the live website and test it in responsive mode, there is no overflow on the x-axis, so I don´t really understand why it shows overflow on the screenshot above.
Best regards, Andrea