[Seaside-dev] WABrush questionable namings

[Lukas Renggli]

These are mostly things that cannot be changed easily, because these
names belong to the most used public API. While I agree that today
some of these attribute-methods would maybe be named slightly
different, you must see that Seaside is collaborative effort of more
than 6 years with more than 30 contributors. It is impossible to have
an absolutely consistent style, even if we try to keep it that way as
much as possible.

> aString = 'columns' ifTrue: [theString := 'cols'].
>  aString = 'selected' ifTrue: [theString := 'checked'].
> aString = 'maxLength' ifTrue: [theString := 'maxlength'].
> aString = 'verticalAlign' ifTrue: [theString := 'valign'].
>  aString = 'character' ifTrue: [theString := 'char'].
> aString = 'characterOffset' ifTrue: [theString := 'charoff'].

I don't understand the above examples.

> url -> src

That makes very much sense, because it makes it possible to handle
images (src) and anchors (href) polymorphic. Attributes should be
named after their intention, not the XHTML definition string.

Cheers,
Lukas

-- 
Lukas Renggli
http://www.lukas-renggli.ch