Re: [patch] <th> around first row elements

From: Vincent McIntyre <>
Date: Sun, 19 Jun 2005 12:20:08 +1000 (EST)
Message-ID: <Pine.SOL.4.33.0506191211300.20408-200000_at_atlas.atnf.CSIRO.AU>

> I think it's fairly arbitrary whether you classify the dates as headers
> or data. I assume that having a visual distinction was the only reason
> for classifying them as headers, and changes which preserve the visual
> distinction in default configurations seem reasonable.
> The class names you propose sound good.
> I will wait until you submit a change that adds the class tags before
> doing anything with the patch you've submitted.

I've gone a little further in adding class names, and verified that this patch WFM. I also corrected an error, before I was putting the class in the <tr> rag, not the relevant <td> tag.

I'd like to take this a bit further still, and remove pretty much all the align= statements in a way that gives CSS fine-grained control over this. But I'll leave that for a later patch, when I have time to give to it; it needs a bit of thought about the page structure. And, of course, some sample CSS files so that the default look is maintained, "out of the box".


Received on Sun 19 Jun 2005 09:18:45 AM GMT

This archive was generated by hypermail 2.2.0 : Thu 22 Feb 2007 07:33:55 PM GMT GMT