@vanzasetia
Posted
Excellent work on this challenge, Yazdun! Your solution looks great! š The transitions on the site obey the prefers-reduced-motion user preference! š
I notice there's a html tag below the <div id="__next" reactroot>. I don't understand Next.js so I can't give you any solution to this problem. š
Also, about the alternative text for the avatar, it would be better if the alternative text is Yazdun - Viking so that the user knows that it's you and you are using the Viking avatar. I think the same as adding the word image or any words that related to it could be verbose because the screen reader is already pronounced it as an image.
I hope this information is useful! Keep up the excellent work! š
Marked as helpful
@Yazdun
Posted
@vanzasetia Hi Vanza, Thanks for your feedback
I actually wasn't aware of html issue before you mentioned it, I was using the html tag in a wrong place, now I fixed it thanks for finding the issue š
Also I think your suggestion about avatar alternative text makes more sense, so I changed that too š
@vanzasetia
Posted
@Yazdun Great job with the update! Keep it up, Yazdun! š
