Hey @pccipri ππ»
I have some suggestions to help you fix the accessibility and HTML issues.
- Pretty much every
<img>
tag in this project, besides user profile pictures, should havearia-hidden="true"
. Like, the following examples:
<img class="bg-top-image-desktop" src="images/bg-pattern-top-desktop.svg" aria-hidden="true">
<img src="images/icon-star.svg" aria-hidden="true">
this should fix all the accessibility and HTML issues. Don't forget to generate a new repot once you fix the issues.
- One small suggestion would be to add some padding to star review boxes.
I hope this was helpful π¨π»βπ» overall, you did a nice job, it looks pretty good on mobile viewport width. Cheers πΎ
Marked as helpful
@pccipri
Posted
@kens-visuals Thanks a lot for the review and the ideas.
@pccipri you're welcome π