Re: Patch for xembed updated

On Fri, Jan 11, 2013 at 06:17:08PM +0100, podarcis wrote:
> 
> > Please have a look at the enclosed diff 'xembed_removed_demo.diff',
> > that removes the *WHOLE* xembed protocol implementation and only
> > keeps the window creation.
> > 
> > Still embedding with 'tabbed' works as before. This really makes me 
> > wonder...
> 
> Update:
> I just tested Claws Mail (writing this mail with it) from the Ubuntu
> 12.04 distribution together with the 'Dillo_Viewer' plugin (which was
> not so easy to get, because it is not longer in this distribution - I
> had to take it from another distibution, openSuse-12.2)
> 
> Result:
> dillo with the *removed xembed support* (see previous posting)
> integrates perfectly (including resizing) into the Claws Mail preview
> window!
> 
> Also does dillo with *my patch*.
> 
> Just to let you know..

I know that it works partly with just simple reparenting, but it has
problems. We went through all this see the thread starting with:
http://lists.auriga.wearlab.de/pipermail/dillo-dev/2009-May/006453.html
and the final success messages from claws-mail people here:
http://lists.auriga.wearlab.de/pipermail/dillo-dev/2009-June/006533.html
I don't know if the claws plugin is still used by anyone, but I
don't want to remove the code, that proved to be working.

Cheers,
Johannes