[carpeperdiem]

Jo,

Before I suggest a solution, please do some housekeeping and update your forum sig with your correct version of xcart -- your fourm sig states, "4.1.18" -- not a valid version of xcart -- is it 4.1.8 or 4.0.18? I will suggest a solution for 4.1.8.

OK -- in /skin1/modules/Bestsellers/menu_bestsellers.tpl

find this code:

{section name=num loop=$bestsellers}
<font class="VertMenuItems"><b>{math equation="value+1" value=$smarty.section.num.index}.</b></font> 
<a href="product.php?productid={$bestsellers[num].productid}&amp;cat={$cat}&amp;bestseller=Y" class="VertMenuItems">{$bestsellers[num].product}</a><br />
{/section}

Notice the {section name=num loop=$bestsellers} -- and the {/section} ???

That's your loop for the items.

Just thinking out loud:

Could you start the <ol> before the loop, and close it after? I think you could. Maybe this:

<ol>
{section name=num loop=$bestsellers}
<li><a href="product.php?productid={$bestsellers[num].productid}&amp;cat={$cat}&amp;bestseller=Y" class="VertMenuItems">{$bestsellers[num].product}</a></li>
 {/section}
</ol>

Notice, I took out the "numbering" function - if you're using a <ul><li> then this is not really needed, is it?: <font class="VertMenuItems"><b>{math equation="value+1" value=$smarty.section.num.index}.</b></font>

I think this will do what you want.

The only drawback is loss of control on the formatting. By doing it the x-cart way, you can use a CSS class to be very specific on linebreaks, etc. Using <ol> may be the better solution?? Please report back! I'm just thinking out loud, so I may be way off here... Hope this helps,

Jeremy