(In reply to comment #49)
> When you submit the form, are all the inputs submitted? Does anything change
> if the form is opened before the table open and closed after the table close?
All the input fields do get submitted, according to the tests I added in the most recent patch.
> In any case, with forms involved like this I can almost see how
> we could switch contexts while the form is still open. Given that, the
> FlushTags is probably what we want.
I think it was worthwhile to drill a little deeper (i.e., into the parser), but I'm also inclined to keep FlushTags in EndContext.
> If you do want to make changes to CNavDTD, then mrbkap is your review man for
> those. I'm willing to review the sink, but CNavDTD is black magic. ;)
(In reply to comment #49)
> When you submit the form, are all the inputs submitted? Does anything change
> if the form is opened before the table open and closed after the table close?
All the input fields do get submitted, according to the tests I added in the most recent patch.
> In any case, with forms involved like this I can almost see how
> we could switch contexts while the form is still open. Given that, the
> FlushTags is probably what we want.
I think it was worthwhile to drill a little deeper (i.e., into the parser), but I'm also inclined to keep FlushTags in EndContext.
> If you do want to make changes to CNavDTD, then mrbkap is your review man for
> those. I'm willing to review the sink, but CNavDTD is black magic. ;)
Don't blame you :)