full stack interactive comments

Community feedback

• 

  Vanza Setia•27,755

  @vanzasetiaPosted about 3 years ago

  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

  0

  ---

  

  Yazdun•1,310

  @YazdunPosted about 3 years ago

  @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 👍

  1

  

  Vanza Setia•27,755

  @vanzasetiaPosted about 3 years ago

  @Yazdun Great job with the update! Keep it up, Yazdun! 👍

  1