Hi,
Im having some issues with looping and applying a list to my menu structure.
I want to group the sub-pages into 5 items with a <ul> tag. So my html would be,

<ul class="nav">
  <li>Heading<li>
  <li>Heading
    <ul class="sub-page">
      <ul class="sub-page-group first-group">
    <li>Item</li>
    <li>Item</li>
        <li>Item</li>
        <li>Item</li>
        <li>Item</li>
      </ul>
      <ul class="sub-page-group second-group">
    <li>Item</li>
    <li>Item</li>
      </ul>
    </ul> 
  <li>

I've had a look through the forum to find something similar and found some xslt, which ive applied.
But im getting some strange html output, with empty <ul> and <li> produced.

Any help would be grateful.

The XSLT that Ive added is:

    <!--DROP DOWN NAVIGATION-->    
    <xsl:if test="*[umbracoNaviHide != 1]">
      <ul>
        <xsl:for-each select="* [position() mod 5 = 1]">
        <ul class="drop-group">
        <xsl:for-each select=". | following-sibling::*[not(position() >= 5)]">
          <li class="drop-item">
            <xsl:value-of select="@nodeName" />
          </li>
        </xsl:for-each>
      </ul>
    </xsl:for-each>  

My full XSLT is

  <xsl:template match="/">  
    <!-- Root Node -->
    <xsl:variable name="rootNode" select="$currentPage/ancestor-or-self::root" />
    <!-- Homepage -->
    <xsl:variable name="homeNode" select="$rootNode/Home [@isDoc]" />    
    <ul>
      <li>
        <!-- Add the CSS class 'selected' if the homeNode ID matches our currentPage node ID
        -->
        <xsl:if test="$homeNode/@id = $currentPage/@id">
          <xsl:attribute name="class">
            <xsl:text>selected</xsl:text>
          </xsl:attribute>
        </xsl:if>       
        <!--Create a link to the homepage -->
        <a href="{umbraco.library:NiceUrl($homeNode/@id)}">
          <xsl:value-of select="$homeNode/@nodeName" />
        </a>
      </li>  
      
    <!--For each child node of the homeNode that is a document (isDoc)and the level is 2 and the property umbracoNaviHide is NOT 1 (true/checked)-->
    <xsl:for-each select="$homeNode/*[@level = 2][umbracoNaviHide != 1]">         
    <li class="{@urlName}">          
    <xsl:if test="$currentPage/ancestor-or-self::*[@isDoc]/@id = current()/@id">
      <xsl:attribute name="class"><xsl:value-of select="concat(@urlName, ' selected')"/></xsl:attribute>
    </xsl:if>
     <xsl:if test="position() = last()">
        <xsl:attribute name="id">
          <xsl:text>last</xsl:text>
        </xsl:attribute>
      </xsl:if>
    <xsl:if test="umbraco.library:HasAccess(@id,@path)">
    <!-- Create the link -->
    <a href="{umbraco.library:NiceUrl(@id)}"><xsl:value-of select="@nodeName" /></a>
    </xsl:if>
    
    <!--DROP DOWN NAVIGATION-->    
    <xsl:if test="*[umbracoNaviHide != 1]">
      <ul>
        <xsl:for-each select="* [position() mod 5 = 1]">
        <ul class="drop-group">
        <xsl:for-each select=". | following-sibling::*[not(position() >= 5)]">
          <li class="drop-item">
            <xsl:value-of select="@nodeName" />
          </li>
        </xsl:for-each>
      </ul>
    </xsl:for-each>        
        
      </ul>
    </xsl:if>    
    </li>
  </xsl:for-each>
  </ul>
  </xsl:template>