On Tue, May 19, 2009 at 08:24:14PM +0200, Johannes Hofmann wrote:
On Tue, May 19, 2009 at 01:34:56PM -0400, Jorge Arellano Cid wrote:
On Mon, May 18, 2009 at 11:14:14PM +0200, Johannes Hofmann wrote:
On Mon, May 18, 2009 at 01:48:43PM -0400, Jorge Arellano Cid wrote:
On Mon, May 18, 2009 at 01:18:11AM +0200, Johannes Hofmann wrote:
Hi,
attached patch tries to implement the missing --xid option that is used to embed dillo into the claws mailer. The right way to do this would probabely be to implement the complete XEMBED protocol which is also mentioned in doc/fltk-problems.doc. However a simple XReparentWindow() seems to work more or less here. Please test.
It seems like you forgot to commit xembed.{hh,cc} to the repo. e.g. hg add xembed.hh xembed.cc
Oh yes. Should be fixed now. Btw. I managed to install the claws mail client including the dillo plugin and it seems to work fine now with dillo2. That's why I just committed the xembed stuff.
This is very good.
I'll have to dig for the sylpheed contact to let them know. Does anybody here know the email?
I already contacted them on irc (#claws on freenode). It would be good if this get's some testing as there seems to be some interaction with the WM in use. I now tested it successful on dwm, twm, and fvwm. Also there are probabely more elegant ways to do it, but for now I think it's better than nothing.
More testing showed that the current implementation has severe problems. The thing is that the window manager that I use (dwm) is one of the few non-reparenting WMs. Most others reparent new windows and that can create races with the reparentin we do. There also seems to be a problem with keyboard events in that case. Is there anyone on the list with in-depth X11 knowledge who could help with the xembed stuff? Cheers, Johannes
Cheers, Johannes