Hi Simon,

> "A quick scan through your stylesheet shows that you're using the
> content of the variable-binding elements quite a lot when you should
> be using the select attribute instead."
>
> I didn't quite understand what you ment by this. Do you mean in the
> template declaration or elswhere?

I meant here:

>     <!-- Get the image-grid-width -->
>     <xsl:variable name="image-grid-width">
>       <xsl:value-of select="@image-grid-width" />
>     </xsl:variable>

and here:

>           <xsl:call-template name="footer-left">
>             <xsl:with-param name="image-nodes">
>               <xsl:value-of select="xalan:nodeset(.)/image[@position =
> 'left']" />
>             </xsl:with-param>
>             <xsl:with-param name="text-nodes">
>               <xsl:value-of select="xalan:nodeset(.)/text[@position =
> 'left']" />
>             </xsl:with-param>
>             <xsl:with-param name="image-grid-width"
> select="$image-grid-width" />
>           </xsl:call-template>

and here:

>           <xsl:call-template name="footer-center">
>             <xsl:with-param name="text-nodes">
>               <xsl:value-of select="xalan:nodeset(.)/text[@position =
> 'center']" />
>             </xsl:with-param>
>           </xsl:call-template>

and here:

>           <xsl:call-template name="footer-right">
>             <xsl:with-param name="image-nodes">
>               <xsl:value-of select="xalan:nodeset(.)/image[@position =
> 'right']" />
>             </xsl:with-param>
>             <xsl:with-param name="text-nodes">
>               <xsl:value-of select="xalan:nodeset(.)/text[@position =
> 'right']" />
>             </xsl:with-param>
>             <xsl:with-param name="image-grid-width"
> select="$image-grid-width" />
>           </xsl:call-template>

But I guess you knew about the latter bunches.

Cheers,

Jeni

---
Jeni Tennison
http://www.jenitennison.com/


 XSL-List info and archive:  http://www.mulberrytech.com/xsl/xsl-list