> ...  and section[ancestor::sale])">

> and the current node - I explicitly named it here but because of the
> match statement in the line above, this need not be - has an ancestor
> called 'sale' 

No, that tests if the current node has a child element section that
has an ancestor sale.

You just want

.... and ancestor::sale

or if you want to check the current node is section, 

... and self::section and ancestor::sale

or similar to your original

... and self::section[ancestor::sale]

David

_____________________________________________________________________
This message has been checked for all known viruses by Star Internet
delivered through the MessageLabs Virus Scanning Service. For further
information visit http://www.star.net.uk/stats.asp or alternatively call
Star Internet for details on the Virus Scanning Service.

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