Group title commit broke display

[prophoto]

It seems this commit broke the display setup of one of my sites. Can this be reversed or tweaked?

https://github.com/Fabrik/fabrik/commit/999b47874ef0e8fae3da19f15f7f0c78d3b1a8c8

"New Exciting Specials" and "Protein Plates" used to display inside of the red bar, now it shows below it.

 

[prophoto]

In case anyone needs this too I created a new layout in /templates/my-template/html/layouts/com_fabrik/lists/fabrik-group-by-heading.php. Basically all I did was move the </a> to the end. If you don't have some of those folders in your template just create them along with the file.

// No direct access
defined('_JEXEC') or die('Restricted access');

$d = $displayData;
$imgProps = array('alt' => FText::_('COM_FABRIK_TOGGLE'), 'data-role' => 'toggle', 'data-expand-icon' => 'icon-arrow-down', 'data-collapse-icon' => 'icon-arrow-right');
?>

<?php if ($d->emptyDataMessage != '') : ?>
<a href="#" class="toggle">
<?php else: ?>
<a href="#" class="toggle fabrikTip" title="<?php echo $d->emptyDataMessage ?>" opts='{"trigger": "hover"}'>
<?php endif; ?>
<?php echo FabrikHelperHTML::image('arrow-down', 'list', $d->tmpl, $imgProps); ?>
<span class="groupTitle">
<?php echo $d->title; ?>
<?php $d->group_by_show_count = isset($d->group_by_show_count) ? $d->group_by_show_count : '1';
if ($d->group_by_show_count) : ?>
<span class="groupCount">(<?php echo $d->count ?>)</span>
<?php endif; ?>
</span>
</a>

 

[cheesegrits]

I've modified that template again.

I'd changed it recently to avoid problems when the group title contained links, and changed the clickable link to only be the up/down arrow. So instead, I'll strip any HTML out of the title.


-- hugh

 

[prophoto]

Thanks Hugh!