[patch] to bug 88243

Paul Temple paul.temple at gmx.net
Sun Feb 13 16:36:36 GMT 2005


Germain Garand wrote:
> > > > Yes, I thought you would say that, but why move to xml/*
> > > > and not dom/* ?
> > >
> > > Because xml/ interferes with the renderer, and never dom/
> > > (which resembles the interface only, and must stay as
> > > lightweight as possible).
> >
> > and because closing the renderer interferes it. Ok, fair
> > enough.
>
> Hi, sorry to intervene that late, but the way I see it, we are
> only closing a node here, not a node's renderer.
> That nodeimpl::close would actually close the renderer is not
> of our concern.
>
> So I rather agree with Koos, that the fix should go in dom/*.
>
> Doing that from xml/* will only get the parser to prematurely
> close nodes while they have children still to be appended.

Doing that in xml - s. today's patch - does not affect the 
parsing process. And parsing <applet>...</applet> already works 
fine, only adding applets via DOM/Script does not work, as 
close / closeRenderer is never called in that case.




More information about the kfm-devel mailing list