@FLCHRIS
Posted
Hi!, congrats on completing this challenge. π₯³π
Here some tips to improve your code. π
Background images are for decorative purposes, they do not add value to your website.
What you can do to fix it is the following, add the aria-hidden='true'
attribute and set the alt
attribute to empty, like so alt=''
. You can also choose to use the background-image
property, thus avoiding using the above.
Example:
<img src="images/pattern-background-desktop.svg" id="img1" alt="" aria-hidden='true' >
<img src="images/pattern-background-mobile.svg" id="img2" alt="" aria-hidden='true' >
I can see that you have a little reset for the default styles that the browser gives, that's ok. But you can improve it by using other resets like the following:
Your solution is very good. π―
Happy coding! βοΈπ
Marked as helpful