forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gav...." <brightoncomput...@brightontown.com.au>
Subject Re: Best way to add repeatable code in v2/v3 (was Re:Exampleusecase)
Date Sat, 28 Jan 2006 05:42:53 GMT

----- Original Message ----- 
From: "Thorsten Scherler" <thorsten.scherler@wyona.com>
To: <dev@forrest.apache.org>
Sent: Friday, January 27, 2006 8:24 PM
Subject: Re: Best way to add repeatable code in v2/v3 (was 
Re:Exampleusecase)


| El vie, 27-01-2006 a las 17:59 +0800, Gav.... escribió:
| > ----- Original Message ----- 
|
| ...
| > |
| > | Actually since Cyriaque and yourself seem to prefer that the default
| > | behavior would be non enclosing divs we can change the default
| > | behavior.
| > |
| > | wdyt?
| >
| > I have found another reason why it is best to close these divs properly.
| >
| > e.g.
| >
| >  <forrest:contract name="content-abstract">
| >             <forrest:properties contract="content-abstract">
| >               <forrest:property name="content-abstract"
| > nugget="get.abstract">
| >                 <jx:import
| > uri="cocoon://#{$cocoon/parameters/getRequest}.abstract.xml"/>
| >               </forrest:property>
| >             </forrest:properties>
| >  </forrest:contract>
| >
| > If there is content to return into this content-abstract request then 
that
| > is fine, on pages where there may not be any content available to 
return,
| > then the contract is blank. This creates a <div class="abstract" /> . It 
so
| > happens that most style sheets I looked at give the abstract div an 
'italic'
| > font-style. Because the div is not closed properly, the rest of the 
content
| > (content-main, content-minitoc-area etc.. ) that follows this 
self-closed
| > div are also outputting the 'italic' font-style.
| >
| > I looked for ages as to why my main content was italic, its because I 
have
| > no abtract-content in the xml source to give.
| >
| > I suppose one has to decide to either use a contract like this for all 
pages
| > or for none.
|
| Why not check whether there is an abstract and if not omit the div?
| <xsl:if test="$content-abstract/div[@class='abstract']/*">
| <xsl:copy-of
|          select="$content-abstract/div[@class='abstract']"/>
| </xsl:if>

You see, this is where I am still catching up with XSL, I didn't think
of solving it that way. Thanks. For the time being though, I moved
my abstract and am using it for something else at the moment. It might
be good to inlcude it somewhere as a commented out example.
Where should it go as an example ?

Gav...

|
| salu2
| -- 
| Thorsten Scherler
| COO Spain
| Wyona Inc.  -  Open Source Content Management  -  Apache Lenya
| http://www.wyona.com                   http://lenya.apache.org
| thorsten.scherler@wyona.com                thorsten@apache.org
|
|
|
|
| -- 
| No virus found in this incoming message.
| Checked by AVG Free Edition.
| Version: 7.1.375 / Virus Database: 267.14.23/242 - Release Date: 
26/01/2006
|
| 



-- 
No virus found in this outgoing message.
Checked by AVG Free Edition.
Version: 7.1.375 / Virus Database: 267.14.23/243 - Release Date: 27/01/2006



Mime
View raw message