@SJ-Nosrat
Posted
Hi, Super solution, pixel perfect (in my opinion)!
-
The webpage is very responsive. The only suggestions I have with is maybe adding some semantic HTML to your HTML structure. As follows:
-
Wrap your
<img src="./images/illustration-hero.svg" alt="">
element in a<figure>
element as this will make your<img>
be contained by a landmark. -
Also, since
<img src="./images/illustration-hero.svg" alt="">
has noalt=""
description, then please add the following atrribute:aria-hidden="true"
, this will allow screenreaders to skip images that are there purely for decorative purposes.
Looking forward to more of your solutions!
Best of luck with your coding journey!
Marked as helpful
@dewslyse
Posted
@shahin1987, thank you very much for the feedback. Really helpful.