<div class="<?=$this->cssClass?>">
    <h2><?=$this->data->trl('Favourites');?></h2>
    <? if ($this->favourites) { ?>
        <? foreach ($this->favourites as $favourite) { ?>
            <div class="favourite">
                <?= $this->componentLink($favourite); ?>
            </div>
        <? } ?>
    <? } else { ?>
        <?=$this->data->trl('You have no favourites!'); ?>
    <? } ?>
</div>

<div class="<?=$this->cssClass?>">
    <h2><?=$this->data->trl('Favourites');?></h2>
    <? if ($this->favourites) { ?>
        <ul>
            <? foreach ($this->favourites as $favourite) { ?>
                <li class="favourite">
                    <?= $this->componentLink($favourite); ?>
                </li>
            <? } ?>
        </ul>
    <? } else { ?>
        <?=$this->data->trl('You have no favourites!'); ?>
    <? } ?>
</div>

Change template to create list
Change template to create list

This is changed because a list of something should use ul and li tags to be a real list also in html
