Hi hbclynch,
Please re-read the thread. The extension was updated to include the image being clickable. You have to update the extension if you have not already and then go back into the module. You will then find a new "Hover" option.
You might actually need to re-read the thread because I don't think you understand the issue. Hovering and clicking on a
text link is different that making an
image clickable and still having the intro text below.
The OP was willing to settle for hover but it's not a good solution for others. We need the actual image to be clickable because that is what people expect when they visit. They know it's leading content and assume that the image should take them to the full article, which it doesn't. Even having to hover and then click the title is counter-intuitive. I had it on a previous site and it was just confusing. People assume that, because the box animates on hover, the whole box is a link. Nope! You still have to click right on the title text.
At any rate, I modified the code to do what we want and it was very easy. Here are the instructions for the benefit of others. It really completes Masonry in my opinion:
Open _item.php in /modules/mod_s5_masonry/tmpl
Copy this from the title link:
<a href="<?php echo $item->link;?>">
Scroll up and paste it before the image_intro img tag that looks like this:
<img alt="" src="<?php echo JURI::root();?><?php echo $images->image_intro; ?>" />
Close the < a > tag after the img tag with this:
</a>
The finished product looks like this:
<div class="s5_masonry_img_wrap">
<a href="<?php echo $item->link;?>">
<img alt="" src="<?php echo JURI::root();?><?php echo $images->image_intro; ?>" />
</a>
</div>
Presto. Actual linking images just like the titles. No hover effect required. Could easily be turned into a setting in the module parameters.
For what it's worth, it would have been great if customer support had directed us to that file for the time being. There's a ton of potential there!