On Fri, Oct 09, 2015 at 11:57:23AM +0200 I heard the voice of
Alexander Klein, and lo! it spake thus:
> Am 24.02.2015 um 22:16 schrieb Rhialto:
> > A quick look at the code shows nothing that pretends to focus the
> > target window. Only to raise it. I'd say it makes sense to focus it
> > as well, in all focus models. I suppose that is an easy enough change
> > to make, unless there is some drawback that I'm missing?
>
> I finally got around to fixing this with an additional line in
> menus.c, see the attached patch.
In poking around and thinking about this, I think it's probably better
to do it in WarpToWindow(), since if it's important in the
WarpAlongRing() case to $USER, it's probably important in any of the
other ways you might warp around too.
So something like the attached (against current head; if you're going
against 3.8.2, you'd probably need to go in manually and add it after
the XWarpPointer() toward the end of WarpToWindow()). If some people
using warping can testdrive it for a bit...
--
Matthew Fuller (MF4839) | [email protected]
Systems/Network Administrator | http://www.over-yonder.net/~fullermd/
On the Internet, nobody can hear you scream.
=== modified file 'menus.c'
--- menus.c 2016-02-08 11:10:48 +0000
+++ menus.c 2016-04-30 20:05:51 +0000
@@ -5205,6 +5205,7 @@ void WarpToWindow(TwmWindow *t, int must
}
}
XWarpPointer(dpy, None, Scr->Root, 0, 0, 0, 0, x + t->frame_x, y + t->frame_y);
+ SetFocus(t, LastTimestamp());
#ifdef DEBUG
{
Window root_return;
