P
Fluffy Kas• 7,735
@FluffyKas
Posted
It's a great looking solution if you ask me, well done! ^^ I'd only add that you could perhaps fix the social links as they're currently not links, only images. Wrap them in <a> tags and give them an aria-label (instead of the alt which is not needed here) and you're good to go.
As an extra note, I'll link this article about alt texts. It's nice that you haven't forgotten about them but they could be improved a bit. (https://axesslab.com/alt-texts/)
Marked as helpful
1
Beeching• 45
@Beeching
Posted
@FluffyKas Ohh, I had totally forgotten about that, tysm!
I just checked out that article and it helped me quite a lot, I really appreciate the tip. Will keep that stuff in mind going forward :) have a good rest of your day!
0
