First of all, just to avoid confusion, the alignment in my patch was
wrong and the correct one would be:
buffer.append("<th align=\"left\"");
Without it the column headers (that are now by default left aligned
because they are rendered with <td>) would be centered in all the
generated lists.
Jacopo
Chris Howe wrote:
My only concern then is why the first change in the
patch _adds the align attribute (it's obvious the rest
of them were already there). I think the "step in the
right direction" is a fine approach and doesn't need
to "fix" everything but it shouldn't throw another
wrench into it. I'm not sure we have enough people to
apply the patch (as opposed to simply reviewing it) to
their development to be sure it's not throwing form
widget uses out of alignment in use cases that use the
areastyle to define their alignment.