change

<xsl:variable name="level" select="2"/>

to

<xsl:variable name="level" select="1"/>

for a start, then see what you get

Don't you just need to include a <li> element for the current page before the for-each statement?

<ul>
  <li><a href="{umbraco.library:NiceUrl($currentPage/@id)}"> <xsl:value-of select="$currentPage/@nodeName"/></a></li>
  ...
</ul>

Hope this helps.

Regards,

/Dirk

Ah, yes - +1 for reading the question correctly :-D

Think you should do it in the following way

<xsl:output method="xml" omit-xml-declaration="yes" />
<xsl:param name="currentPage"/>
<xsl:variable name="level" select="2"/>
<xsl:template match="/">

<!-- The fun starts here -->

<ul>

<!-- the new code that displays link to level 2 page -->

<xsl:variable name="level2page" select="$currentPage/ancestor-or-self::node [@level=$level]"/>

 <li>
  <a href="{umbraco.library:NiceUrl($level2page/@id)}">
  <xsl:if test="$currentPage/ancestor-or-self::node/@id = $level2page/@id">
  <xsl:attribute name="style">font-weight: bold;</xsl:attribute>
  </xsl:if>
  <xsl:value-of select="$level2page/@nodeName"/>
  </a>
 </li>

<!-- the new code ends-->


<xsl:for-each select="$currentPage/ancestor-or-self::node [@level=$level]/node[string(data [@alias='umbracoNaviHide']) != '1']">
 <li>
  <a href="{umbraco.library:NiceUrl(@id)}">
  <xsl:if test="$currentPage/ancestor-or-self::node/@id = current()/@id">
  <!-- we're under the item - you can do your own styling here -->
  <xsl:attribute name="style">font-weight: bold;</xsl:attribute>
  </xsl:if>
  <xsl:value-of select="@nodeName"/>
  </a>
 </li>
</xsl:for-each>
</ul>

</xsl:template>

Hope this will help.

BTW you have redudant add of attribute style="font-weight:bold" becouse of your css defines it as bold always.

2 Dirk

I think the task is not to display link to the current page but the link to the parent page of the pages listed in subnavigation.

You're right Vladimir, must have misunderstood, indeed, needs to have link to parent page only, no matter what child node page he's on...

On the other hand (talking to inmedia again), why repeat that page if that one is already marked as selected section  in top navigation?

Cheers,

/Dirk

Agree with you Dirk about non-logical requirements. But sometimes customer wants wired things =)

I think this link to the parent page at least should have a different look.

up to inmedia to educate the customer then :p

Hi Vladimir

I think your answer might be the solution... But... (And here comes a really stupid question, I guess)... Do I need to replace the "$level2page" with something? Cause I get an error trying to save it as you posted it.

And yes, I agree that it is a non-logical requirement, but the customer is (almost) always right :)

No one???.....

Error is probably due to fact that $level2page doesn't have a value until runtime, but NiceUrl expects a int parameter... Either surround with xsl:if statement or tick the 'Ignore errors' checkbox.

Does it render ok at the front-end?

Cheers,

/Dirk

Hi Dirk

I ticked the "ignore error" and it works and renders perfect... Thank you for helping out a rookie :)

What kind of error do you have?

Oops, error is resolved. Don't forget to mark a post as a solution ;)

is working on a reply...