forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gav...." <>
Subject Re: Best way to add repeatable code in v2/v3 (was Re: Exampleusecase)
Date Fri, 27 Jan 2006 09:59:12 GMT

----- Original Message ----- 
From: "Thorsten Scherler" <>
To: <>
Sent: Sunday, January 22, 2006 9:56 PM
Subject: Re: Best way to add repeatable code in v2/v3 (was Re: 

<snip >

| > | <forrest:hook name="headlines"></forrest:hook>
| > | creates
| > | <div id="headlines />
| > |
| > | and <forrest:hook name="headlines" nbsp="true"></forrest:hook>
| > | gives
| > | <div id="headlines"> </div>
| > |
| > | We need to document this in the howto, but self enclosing divs are
| > | default in the dispatcher, no bug!
| >
| > Thats fine, I will send in a patch to document this. I can not really
| > think of a situation (except in testing) where I would want an
| > empty div anyway, but came across it so mentioned it.
| 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.


 <forrest:contract name="content-abstract">
            <forrest:properties contract="content-abstract">
              <forrest:property name="content-abstract" 

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.


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

View raw message