Re: more TRST support (was: Re: lynx-dev another largish patch)

[Klaus Weide]

On Wed, 27 Oct 1999, Leonid Pauzner wrote:

> 26-Oct-99 15:53 Klaus Weide wrote:
> 
> > Currently nothing is done for THEAD, TFOOT, TBODY.  Actually, they cancel
> > TRST processing:

Well I was wrong, misread my own code...
HText_endStblTR only ends the current row, it is not an HText_cancelStbl.

> >     case HTML_THEAD:
> >     case HTML_TFOOT:
> >     case HTML_TBODY:
> >         HText_endStblTR(me->text);
> 
> Hmmm. They should be ignored at least...

The client is supposed to show TFOOT rows not where they occur in the
input document but as a footer that may be repeated; and similar for
THEAD.  I think I skipped dealing with them completely because we
aren't doing anything of that sort.

But TRST isn't goin to make things worse, so that isn't really a good reason
for anything.

> > Anyway, such incremental improvements to TRST stuff shouldn't distract
> > from the fact that it isn't full table support...  Effort may be better
> > spent adding some fuller table support, rather than implementing obscure
> > ways to set alignment for TRST if those aren't used in practice anyway.

And having said that, nobody has to listen to me, not even myself...

New patch, temporarily available under
    <http://enteract.com/~kweide/lynx/>
as
      col-tbody-etc.diff

* fixes an invalid memory problem from previous patch
* adds alignment inheritance from COLGROUP / COL / THEAD / TFOOT / TBODY.


    Klaus