Thanks a lot. I think I get it now.

Ming

>
> You have
>  <xsl:choose>
>     <xsl:when test="idlink">
>      <xsl:copy-of select="idlink[1]/preceding-sibling::node()"/>
>       <xsl:for-each select="idlink">
>
>
> That says, if the curent node has an idlink child anywhere, first copy
> all the children after the first idlink (including any later idlinks)
> then after that process all the idlink elements.
>
>
> You don't want to do that at all, you just want to process each element
> in the order it appears in the file, so replace all of the code you showed
> by a single
> <xsl:apply-templates/>
> then have a template for idlink that makes a link,as has been shown
> earlier.
>
> David
>
> ________________________________________________________________________
> This e-mail has been scanned for all viruses by Star. The
> service is powered by MessageLabs. For more information on a proactive
> anti-virus service working around the clock, around the globe, visit:
> http://www.star.net.uk
> ________________________________________________________________________