forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ferdinand Soethe <>
Subject Re: Cleaning up html-processing
Date Thu, 06 Oct 2005 06:42:55 GMT

Hi Kevin,

thanks for taking the time to explain.
Re-reading a forth time I now understand that with

> I think should be:
>  <xsl:when test="not(@class)">

you actually suggested part of the fix that David later committed.
I'm sorry. I just saw 'not' and your explanations below and concluded
that you wanted to turn around my conditions. Should have looked at
the code a little closer.

> is not needed as the original version worked fine. I wasn't
> suggesting any new approach.

Well in that point I still disagree. In order for the old version
below to work ok you have to overwrite _all_ the attributes that are set
or you'll end up with some attributes remaining unchanged.

> <xsl:template match="table">
>     <xsl:apply-templates select="@id"/>

>     <table cellpadding="4" cellspacing="1" class="ForrestTable">
>           <xsl:copy-of select="@cellspacing | @cellpadding |
> @border | @class | @bgcolor |@id"/>
>           <xsl:apply-templates/>
>     </table>

My goal however was to provide a clean alternative for css-formatting
and that meant I wanted _no_ spacing and padding attributes in the

So I still believe that the new solution is cleaner and better than
the old code.

> so now what worked is partly redundant. Sorry I can't explain,
> I'm doubting myself now.

If I once again missed something, let me know.

Ferdinand Soethe

View raw message