P
Matt Studdert• 13,611
@mattstuddert
Posted
Great job on this challenge! I'd recommend sticking to just a single h1 on the page, as having multiple can confuse the content hierarchy for screen reader users. Also, for the social links, I'd recommend adding an aria-label attribute to the anchor tags, otherwise screen reader users won't know what the links are for. This is because of the lack of text inside the anchor tag.
Keep up the great work! 🙂
0
P
Matt Studdert• 13,611
@mattstuddert
Posted
No problem!
0
tsv• 130
@tsvGithub
Posted
Hello Matt! Thank you very much for your feedback. I appreciate your guidance. I've just changed page structure to have only one 'h1' and also added 'aria-label' to social links. Thank you.
0
