Row Alternation

[Damien Cassou]

On Tue, Nov 18, 2008 at 12:28 AM, David Farber <rdrvr.biz at gmail.com> 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