Row Alternation

> On Tue, Nov 18, 2008 at 12:28 AM, David Farber <[hidden email]> wrote:
>> Here's a little change set I whipped up today.  It annotates Table rows with
>> 'class=odd' or 'class=even' so that you can apply css to color alternate
>> rows.  Comments, criticisms, etc., welcome.
>
> Part of your code reads:
>
> visitTableRow: anObject
>        html tableRow
>                class: (anObject rowAlternation isNil ifFalse: [ anObject rowAlternation ]);
>                with: [ super visitTableRow: anObject ]
>
>
> class: (anObject rowAlternation isNil ifFalse: [ anObject rowAlternation ])
>
> seems to be equivalent to:
>
> class: anObject rowAlternation
>
> which seems wrong because you will add a nil class in some cases.
>
> What about:
>
> html tableRow
>                class: anObject rowAlternation if: anObject rowAlternation notNil
>
>
> --
> Damien Cassou
> http://damiencassou.seasidehosting.st
>
> _______________________________________________
> SmallWiki, Magritte, Pier and Related Tools ...
> https://www.iam.unibe.ch/mailman/listinfo/smallwiki
>

> On Tue, Nov 18, 2008 at 12:28 AM, David Farber <[hidden email]>  
> wrote:
>> Here's a little change set I whipped up today.  It annotates Table  
>> rows with
>> 'class=odd' or 'class=even' so that you can apply css to color  
>> alternate
>> rows.  Comments, criticisms, etc., welcome.
>
> Part of your code reads:
>
> visitTableRow: anObject
> html tableRow
> class: (anObject rowAlternation isNil ifFalse: [ anObject  
> rowAlternation ]);
> with: [ super visitTableRow: anObject ]
>
>
> class: (anObject rowAlternation isNil ifFalse: [ anObject  
> rowAlternation ])
>
> seems to be equivalent to:
>
> class: anObject rowAlternation
>
> which seems wrong because you will add a nil class in some cases.