this variant is known to work, but focus() is ugly - we need in general a better way to handle multihead, this issel-stuff looks awkward (maybe it might be a good idea to set sel to NULL but to introduce a Client *revert which is set if a screen is unfocused, have to think about it further).
--- a/client.c Mon Jan 22 10:22:58 2007 +0100
+++ b/client.c Mon Jan 22 10:35:58 2007 +0100
@@ -81,7 +81,7 @@
focus(Client *c) {
Client *old = sel;
- if(!issel || (c && !isvisible(c)))
+ if(c && !isvisible(c))
return;
if(old && old != c) {
@@ -89,14 +89,18 @@
XSetWindowBorder(dpy, old->win, dc.norm[ColBorder]);
}
if(c) {
- detachstack(c);
- c->snext = stack;
- stack = c;
- grabbuttons(c, True);
- XSetWindowBorder(dpy, c->win, dc.sel[ColBorder]);
- XSetInputFocus(dpy, c->win, RevertToPointerRoot, CurrentTime);
+ if(issel) {
+ detachstack(c);
+ c->snext = stack;
+ stack = c;
+ grabbuttons(c, True);
+ XSetWindowBorder(dpy, c->win, dc.sel[ColBorder]);
+ XSetInputFocus(dpy, c->win, RevertToPointerRoot, CurrentTime);
+ }
+ else
+ XSetWindowBorder(dpy, c->win, dc.norm[ColBorder]);
}
- else
+ else if(issel)
XSetInputFocus(dpy, root, RevertToPointerRoot, CurrentTime);
sel = c;
drawstatus();
--- a/event.c Mon Jan 22 10:22:58 2007 +0100
+++ b/event.c Mon Jan 22 10:35:58 2007 +0100
@@ -269,8 +269,8 @@
XCrossingEvent *ev = &e->xcrossing;
if((ev->window == root) && !ev->same_screen) {
- focus(NULL);
issel = False;
+ focus(sel);
}
}