Hi Eddie,

what if you remove the recursive call from the match="node" template (aka get rid of this bit:

      <!-- Continue down this branch? -->
      <xsl:if test="descendant-or-self::node = $currentPage and node[data[@alias = 'umbracoNaviHide'] = 0]">
        <ul>
          <xsl:apply-templates select="node" />
        </ul>
      </xsl:if>

and start the call one level deeper or at a specific node? Not sure if I understand you correctly though.

Sascha

Hi Sascha

Unfortunately removing that section gives the opposite effect and produces:

<ul>
    <li class="current">
        <a href="/about.aspx">About Us</a>
    </li>
</ul>

It looks like I need to retain this part, and possibly call a second template. The following XSLT produces the correct mark-up:

  <xsl:param name="currentPage"/>

  <xsl:template match="/">
    <xsl:call-template name="construct">
      <xsl:with-param name="node" select="$currentPage/ancestor-or-self::node [@level = 2]"/>
    </xsl:call-template>
  </xsl:template>

  <xsl:template name="construct">
    <xsl:param name="node"/>
    <xsl:param name="id"/>
    <xsl:if test="$node/node [string(./data [@alias='umbracoNaviHide']) != '1']">
      <ul>
        <xsl:for-each select="$node/node [string(./data [@alias='umbracoNaviHide']) != '1']">
          <li>
            <xsl:if test="$currentPage/@id = current()/@id">
              <xsl:attribute name="class">current</xsl:attribute>
            </xsl:if>
            <a href="{umbraco.library:NiceUrl(@id)}">
              <xsl:value-of select="@nodeName"/>
            </a>
            <xsl:if test="count(./node) &gt; 0">
              <xsl:call-template name="construct">
                <xsl:with-param name="node" select="."/>
              </xsl:call-template>
            </xsl:if>
          </li>
        </xsl:for-each>
      </ul>
    </xsl:if>
  </xsl:template>

</xsl:stylesheet>

But I cannot work out how to get this to only show the child menu items related the current page.  So when people is the current page, the menu would look like this:

            <ul>
                <li><a href="/about/people.aspx">People</a>
                    <ul>
                        <li>...</li>
                        <li>...</li>
                    </ul>
                </li>
                <li><a href="/about/history.aspx">Philosophy</a></li>
            </ul>

Or when History is the current page then the menu would look like:

            <ul>
                <li><a href="/about/people.aspx">People</a></li>
                <li><a href="/about/history.aspx">History</a>
                    <ul>
                        <li>...</li>
                        <li>...</li>
                    </ul>
                </li>
            </ul>

Ideally I need the mark-up form the xslt in this thread, but with the show/hide effect produced from by the xslt in the opening message.

Any ideas on how to achive this?

Thanks again

Eddie

Hi Eddie,

I think I understand it now. Basically the items at the first level will always be the same, e.g.

people
history
something else
...

but you only want the sub tree to expand for the page people are currently on, so if I'm on the history page I'll get

people
history
- founding fathers
- first steps
- ...
something else
...

You should be able to achieve that by extending the if test before you call the sub nodes, so instead of

<xsl:if test="count(./node) &gt; 0">

do something like

<xsl:if test="count(./node) &gt; 0 and ./@id = $currentPage/@id">

So basically build the child nodes if
- there are any and
- the id of the first level node the loop is currently processing is identical to the id of the current page.

Hope it works,
Sascha

Hi Sascha

It almost works, the correct sub pages are shown when clicking on history.  But when I then click on say founding fathers, it only shows people and history.  When I need to show:

history
- founding fathers
  -plus any child pages
- first steps

I had a similar thinking to you, where I would add a count and when greater then 0, create the menu.  But I was thinking of using it with the xslt at the beginning of the thread.  But to date, have not got it to work.  Will keep trying.

Thanks again for you help.

Eddie

is working on a reply...