P
Robert Crocker• 410
@robcrock
Posted
The HTML is nice and easy to follow. My only feedback there is that you could've have used a string template to make the alt text more specific. The anchor tags could have also been put in an unordered list and some dummy links could have been added to those anchor tags.
Marked as helpful
1
Eugene Amedior• 140
@Eugene-ak
Posted
@robcrock I appreciate the suggestions and will refactor accordingly but I don't get what you mean with the alt text.
0
P
Robert Crocker• 410
@robcrock
Posted
@Eugene-ak sure 👍 what I meant there was just a userName variable to something like "Example Name", then the alt text could receive alt={${userName}'s profile image}
Marked as helpful
1
Eugene Amedior• 140
@Eugene-ak
Posted
Oh okay. That's a great idea. Thanks @robcrock
0
