Social-proof section try with flexbox
Design comparison
Community feedback
- @grace-snowPosted almost 4 years ago
Hi Sebastian
This looks ace!!
Only issue is with your alt text at the moment :
- on the stars it's important the alt text is kept empty, so assistive technology knows to skip over those images and not announce them
- in general, alt text should never say things like 'photo of' or 'image of' because it is already being announced as an image before the alt text is read out. It needs to be a meaningful description
- if the html lang and content is in English, the alt text needs to be too I'm afraid.
Maybe you should translate all the other content into Spanish thoigh (sorry if I've got the wrong language there! 🙈). You could then change the html lang and teach us all how this challenge would look and read in another language - we might all learn from you and there is no reason why the FM challenges need to be completed in English ☺
I hope this is a helpful comment. Keep going with your coding
1@AngelGris76Posted almost 4 years ago@grace-snow Thank you very much for your feedback. Is the "alt" attributes only important for certain images. In profile photo, can i omit the "alt" attribute. Or just i put the user name
0@grace-snowPosted almost 4 years ago@AngelGris76 you always need the alt attribute, but if it's not a meaningful image (decorative for example) it should be
alt=""
so that assistive tech knows it has been left intentionally empty0@grace-snowPosted almost 4 years agoOn the profile images, I would say they are meaningful, so yes the persons name would be appropriate for that alt attribute
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord