Mailing-List: contact forrest-dev-help@xml.apache.org; run by ezmlm
Precedence: bulk
Reply-To: forrest-dev@xml.apache.org
Subject: Re: [PATCH] to the book2menu in forrest skin
From: David Crossley <crossley@indexgeo.com.au>
To: forrest-dev@xml.apache.org
In-Reply-To: <3F5EE57B.6040807@che-che.com>
References: <3F5BD97D.6040806@verizon.net> <3F5D0D00.6080602@che-che.com>
	 <20030908232837.GC65879@minotaur.apache.org>
	 <1063067305.32157.6858.camel@ighp> <3F5D21DE.4050207@che-che.com>
	 <bjlu8a$mg$1@sea.gmane.org>  <3F5E80F1.5000209@che-che.com>
	 <1063161572.32156.15491.camel@ighp>  <1063162470.32157.15572.camel@ighp>
	 <1063164477.32156.15736.camel@ighp>  <3F5EDFF0.1090607@che-che.com>
	 <1063182971.32157.17401.camel@ighp>  <3F5EE57B.6040807@che-che.com>
Content-Type: text/plain
Organization: 
Message-Id: <1063248699.32157.23793.camel@ighp>
Mime-Version: 1.0
Date: 11 Sep 2003 15:38:19 +1000
Content-Transfer-Encoding: 7bit

Juan Jose Pablos wrote:
> David,
> 
> I just check the dtd and the comments from site2book.xsl are wrong, plus 
> we need to handle the type attribute as well.
> 
> A menu-item is considerer for display if the type is not hidden.
> 
> Is that dtd the official one?

Yes, it is a copy of the one from Cocoon CVS. As i said, it can be
changed, but that does not really help much because we still need
to maintain back-compatibility for the type=hidden.
--David

> Cheers,
> Cheche
> 
> David Crossley wrote:
> > Juan Jose Pablos wrote:
> > 
> >>David Croslsey wrote:
> >>
> >>>Oops, i had better keep quiet today. I take that back about being able
> >>>to remove the label attribute from book.xml menu-items - you cannot. 
> >>
> >>can you elaborate a bit more of why you can not remove @label from a 
> >>book.xml?
> > 
> > 
> > DTD validation error, @label is required. We could change
> > that i suppose, as making something optional would not
> > bust any back-compatibility.
> > 
> > However, we would still need to maintain handling anyway,
> > of the type=hidden thing.
> > 
> > --David
> >