Re: Re: HTML_Table: thead, tfoot and tbody

Helgi Þormar wrote: >> No response from the maintainers since one month ... > > We have a lot to do :-) No problem, as this is not an urgent request. It should just be an information for Christian that there is already a feature request in the bug tracker. > If you are still ready to make the patch as you say in the bug report > than we welcome it. I can do it, no problem. But there should be an agreement on which way to do it. Christian is right that my suggestion can get confusing when adding more data to the body of the table. I really like his second suggestion (using a fourth parameter in setCellContents()). Therefore, my new suggestion: - add a parameter to toHtml() for keeping BC compatibility: function toHtml(boolean $withTParts = false); ==> only with "$withTParts === true" <thead>, <tfoot> and <tbody> will be generated - add a fourth parameter to setCellContents(): function setCellContents(int $row, int $col, mixed $contents [[, string $type='TD'], boolean $tbody = true]); ==> "normal" data belongs to <tbody>, new parameter allow to add data to <tfoot> - content added by setHeaderContents() belongs to <thead> Opinions? Mark