Shahin NJ• 1,190
@SJ-Nosrat
Posted
Hi Luis, Great solution! Looks wonderful.
I'd only suggest adding the svg images as <img aria-hidden="true" src="./img/bg-pattern-bottom-desktop.svg" alt="bg-pattern-bottom-desktop.svg" class="bg-pattern-bottom-desktop"> instead adding the raw <svg>.
Marked as helpful
1
Luis• 930
@luis08201
Posted
Hi @shahin1987, thanks for the feedback. I already reviewed the code and made the changes you mention. Regards.
1
Shahin NJ• 1,190
@SJ-Nosrat
Posted
@luis08201 Great! Just one last thing; don't forget the aria-hidden="true" attribute to add to the <img> this will allow screen readers to skip images that are purely decorative; it'll really be a bad user experience to have them tab through every star on the page.
Best of luck!
Marked as helpful
0
