Topic-icon Alt tag for Facebook stream

Active Subscriptions:

None
9 years 3 weeks ago #61766 by picht
Hi

I run my website through a validator and it comes up with this "error":
img src="https://scontent.xx.fbcdn.net/v/t1.0-1/p320x320/12729011_1738449966387225_2519115353245019165_n.png?oh=09e8cae99758e05bab6070921df617aa&oe=59BEBB74" /></div>

An “img” element must have an “alt” attribute, except under certain conditions. For details, consult guidance on providing text alternatives for images.


Is it possible to add this tag or can it be added by default in the component?

Regards
Picht
The topic has been locked.
Support Specialist
9 years 3 weeks ago #61774 by mel
Replied by mel on topic Alt tag for Facebook stream
For the SCSocialStream module, you can find where we're adding the images in the /media/sourcecoast/themes/scsocialstream/default/facebook.php file. There are 4 instances of the img tags. Could you try adding an alt attribute like the following to see if that helps resolve your issue:

Line 19:
echo '<img src="' . JUri::root(true) . '/media/sourcecoast/images/provider/facebook/icon.png" alt="Facebook Login" />';

Line 25:
<div class="author-image"><img src="'.JFBCFactory::provider('facebook')->profile->getAvatarUrl($this->authorID,false,null).'" alt="Author"/></div>

Line 58:
echo '<div class="image"><img src="' . $this->thumbPicture . '" alt="Post Thumbnail"/></div>';

Line 81:
<span class="from text-left"><img src="https://graph.facebook.com/' . $comment['from']['id'] . '/picture" width="25" alt="Comment Thumbnail"/> ' . $comment['from']['name'] . '</span>

Of course if you're referring to something other than the SCSocialStream module please provide more details/a specific URL where I can see the tags. Let me know if that helps and I can easily check this in for the next release.

-Melissa
The topic has been locked.
Active Subscriptions:

None
9 years 3 weeks ago #61779 by picht
Replied by picht on topic Alt tag for Facebook stream
That was exactly what i needed.
Please incorporate it in the next release if possible.

Thanks!
The topic has been locked.
Support Specialist
9 years 3 weeks ago #61780 by mel
Replied by mel on topic Alt tag for Facebook stream
Will do. I've checked in the code already.
The topic has been locked.