forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Juan Jose Pablos <che...@che-che.com>
Subject Re: [PATCH] to the book2menu in forrest skin
Date Wed, 10 Sep 2003 08:20:54 GMT
David,

I am sorry if I did not express myself clear enough, It was too late I 
guess.

I wanted to give users a hint of where to use this new type and add this 
  new type on to a menu on the fresh-site Then I found this on 
site2book.xsl:

       <!-- No label, abandon the whole subtree -->
       <xsl:when test="not(@label)">
       </xsl:when>
       <!-- Below here, everything has a label, and is therefore 
considered "for display" -->

       <!-- No children -> must be a menu item -->
       <!-- Has children, but they are not for display -> menu item -->

So I drop an e-mail to verify with Vadim if when on book.xml you have 
not got a label on a menu item gives the same behaviour.

David Crossley wrote:
> 
> Cocoon does not (yet) have a site.xml for their docs, so we need
> to ensure that the old book.xml behaviour works properly. It does,
> with Vadim's patch.
> 

I did not know where it was going to be applied.

> By the way, i tested another project that does have a site.xml
> and the approach with missing label attr does work too. Nice.

so if produce the same result we should be using only one approach: 
Either @label or @type="hidden"



Cheers,
Cheche


Mime
View raw message