Re: Re: HTML_Table: thead, tfoot and tbody
| From: | Klaus Guenther | Date: | Sun, 31 Jul 2005 21:07:48 +0000 |
| Subject: | Re: Re: HTML_Table: thead, tfoot and tbody | ||
| References: | 1 2 3 4 5 6 7 8 9 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-39065@lists.php.net to get a copy of this message | ||
On 7/31/2005 4:30 AM, Helgi Þormar spake the following words:
> On Sat, 2005-07-30 at 23:59 +0200, Mark Wiesemann wrote:
>>Christian Weiske wrote:
>>>>How about allowing the $row and $col params in
>>>>set{Header/Body/Footer/Cell}Contents() to be null? E.g.: If $col = null
>>>>then the content should go in every col of the specified row.
>>>>Accordingly for the cases $row = null and ($row = null && $col = null).
>>>
>>>Yes, that is a quite neat solution.
>>>Only: Where to pass the offsets?
>>
>>I talked to Christian off-list: my suggestion with the null values was
>>based upon a missunderstanding.
>>
>>So my new suggestion (agreed by Christian) is to use $col as the offset
>>value if $contents in set{Header/Body/Footer/Cell}Contents() is an
>>array. Example:
>>
>>$table->setCellContents(3, 2, array('A', 'B', 'C'));
>>==> row 3, col 2 = 'A'
>> row 3, col 3 = 'B'
>> row 3, col 4 = 'C'
>
> This sounds nice, plus the other things said before, I'm going out of
> town for 3-4 days in couple of hours so I can't do anything then please
> email me the patch + doc changes (well the doc changes can go to the doc
> list after the patch is added really) when you have it ready :)
+1 here, too :) I've been lurking, even if I haven't said anything on
the subject just yet. That sounds like a good implementation. I'd have
probably thought of adding an additional instance of HTML_Table to the
object for the header and the footer, but if we can do without it, it
would save time and memory.
Klaus