Vanza Setia• 27,795
@vanzasetia
Posted
Hi, Sergio! 👋
Congratulations on finishing this challenge! 👏
I have a few suggestions for this solution.
- I would recommend removing the
aria-label
from the anchor tag that already has text content. I am afraid that it might make the screenreader pronounce both of them which can make users get annoyed. - I recommend adding
rel="noopener"
to any anchor tags that havetarget="_blank"
. This is a security essential for external links. I suggest reading the web.dev article to learn more about this. - Lastly, I suggest fixing all the issues that have been reported.
That's it! I hope you find this useful! 😊
Marked as helpful
1