PUG+ PopUp Utility & Gallery

Advanced Search

HTML output of gallery thumbnail grid doesn't validate

Permalink Browser Info Environment
When i'm trying to use your add-on for gallery with thumbnail grid, there is problem with validation of the output: the thumbnails images are lacking the "alt" attribute as well as the closing image tag.
I would like to correct this at least within a custom template, but i don't know how to achieve that. Could you please give me a hint?
Type: Discussion
Status: New
okapi
View Replies:
okapi replied on at Permalink Reply
okapi
Now i succeeded in cleaning up the html a little bit. It's all in controller.php and obviously that file cannot get overridden in the root blocks directory, so i had to apply the changes to the original controller.php in the add-on package.

Maybe the author could implement these small tweaks in future versions.

controller.php

line 25:
replace
$v->addHeaderItem('<link rel="stylesheet" type="text/css" href="' . $uh->getBlockTypeAssetsURL($bt) . '/fancybox/jquery.fancybox-1.3.4.css">','CONTROLLER');

with
$v->addHeaderItem('<link rel="stylesheet" type="text/css" href="' . $uh->getBlockTypeAssetsURL($bt) . '/fancybox/jquery.fancybox-1.3.4.css" />','CONTROLLER');


line 456:
replace
$displayContent .= '<img src="' . $thumbnail->src . '"></a> </div>';

with
$displayContent .= '<img src="' . $thumbnail->src . '" alt="" /></a> </div>';


line 466:
replace
$displayContent .= '</table>';

with
$displayContent .= '</td></tr></table>';
pvernaglia replied on at Permalink Reply
pvernaglia
Hi, I will look into fixing that in the next release. Thanks for pointing it out

concrete5 Environment Information

Browser User-Agent String

Hide Post Content

This will replace the post content with the message: "Content has been removed by an Administrator"

Hide Content

Request Refund

You have not specified a license for this support ticket. You must have a valid license assigned to a support ticket to request a refund.