You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@forrest.apache.org by Thorsten Scherler <th...@apache.org> on 2006/10/20 16:50:03 UTC

Re: svn commit: r466066 - /forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft

On Fri, 2006-10-20 at 11:19 +0000, cdupoirieux@apache.org wrote:
> Author: cdupoirieux
> Date: Fri Oct 20 04:19:45 2006
> New Revision: 466066
> 
> URL: http://svn.apache.org/viewvc?view=rev&rev=466066
> Log:
> Need to pass the context here...

Hmm, why?
<xsl:stylesheet version="1.1"
xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
      <xsl:param name="content-main-conf" select='boxed'/>

Meaning all xsl:templates "know" $content-main-conf. No need to ...

> 
> Modified:
>     forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft
> 
> Modified: forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft
> URL: http://svn.apache.org/viewvc/forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft?view=diff&rev=466066&r1=466065&r2=466066
> ==============================================================================
> --- forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft (original)
> +++ forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft Fri Oct 20 04:19:45 2006
> @@ -95,8 +95,12 @@
>        </xsl:template>
>  
>        <xsl:template match="@*|*|text()|processing-instruction()" priority="-1">
> +        <xsl:param name="content-main-conf"/>

see above. 

IMO we should remove all <xsl:param name="content-main-conf"/> in the
xsl:template.

salu2
-- 
thorsten

"Together we stand, divided we fall!" 
Hey you (Pink Floyd)


Re: svn commit: r466066 - /forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft

Posted by Thorsten Scherler <th...@apache.org>.
On Mon, 2006-10-23 at 09:37 +0200, Cyriaque Dupoirieux wrote:
> le 20/10/2006 16:50 Thorsten Scherler a écrit :
> > On Fri, 2006-10-20 at 11:19 +0000, cdupoirieux@apache.org wrote:
> >   
> >> Author: cdupoirieux
> >> Date: Fri Oct 20 04:19:45 2006
> >> New Revision: 466066
> >>
> >> URL: http://svn.apache.org/viewvc?view=rev&rev=466066
> >> Log:
> >> Need to pass the context here...
> >>     
> >
> > Hmm, why?
> > <xsl:stylesheet version="1.1"
> > xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
> >       <xsl:param name="content-main-conf" select='boxed'/>
> >
> > Meaning all xsl:templates "know" $content-main-conf. No need to ...
> >   
> Yes, I think your are right, because I have made some tests friday night.
> I am going to continue my tests and correct this.
> 

Cheers, Cyriaque.

> Thanks,

to you.

salu2

> 
> Salutations,
> Cyriaque,
> 
> >   
> >> Modified:
> >>     forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft
> >>
> >> Modified: forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft
> >> URL: http://svn.apache.org/viewvc/forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft?view=diff&rev=466066&r1=466065&r2=466066
> >> ==============================================================================
> >> --- forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft (original)
> >> +++ forrest/trunk/whiteboard/plugins/org.apache.forrest.themes.core/themes/pelt/html/content-main.ft Fri Oct 20 04:19:45 2006
> >> @@ -95,8 +95,12 @@
> >>        </xsl:template>
> >>  
> >>        <xsl:template match="@*|*|text()|processing-instruction()" priority="-1">
> >> +        <xsl:param name="content-main-conf"/>
> >>     
> >
> > see above. 
> >
> > IMO we should remove all <xsl:param name="content-main-conf"/> in the
> > xsl:template.
> >
> > salu2
> >   
-- 
thorsten

"Together we stand, divided we fall!" 
Hey you (Pink Floyd)