Xinqi Bao's Git

projects / dwm.git / blobdiff
summary | log | commit | diff | tree
raw | inline | side by side
several other focus fixes, introduced unfocus()
[dwm.git] / dwm.c
diff --git a/dwm.c b/dwm.c
index 1909d4b..d3169be 100644 (file)
--- a/dwm.c
+++ b/dwm.c
@@ -211,6 +211,7 @@ static void togglebar(const Arg *arg);
 static void togglefloating(const Arg *arg);
 static void toggletag(const Arg *arg);
 static void toggleview(const Arg *arg);
 static void togglefloating(const Arg *arg);
 static void toggletag(const Arg *arg);
 static void toggleview(const Arg *arg);
+static void unfocus(Client *c);
 static void unmanage(Client *c);
 static void unmapnotify(XEvent *e);
 static void updategeom(void);
 static void unmanage(Client *c);
 static void unmapnotify(XEvent *e);
 static void updategeom(void);
@@ -391,9 +392,23 @@ buttonpress(XEvent *e) {
        unsigned int i, x, click;
        Arg arg = {0};
        Client *c;
        unsigned int i, x, click;
        Arg arg = {0};
        Client *c;
+       Monitor *m;
        XButtonPressedEvent *ev = &e->xbutton;
 
        click = ClkRootWin;
        XButtonPressedEvent *ev = &e->xbutton;
 
        click = ClkRootWin;
+       /* focus monitor if necessary */
+       for(m = mons; m; m = m->next)
+               if(ev->window == m->barwin) {
+                       if(m != selmon) {
+                               if(selmon->stack)
+                                       focus(selmon->stack);
+                               else {
+                                       selmon = m;
+                                       focus(NULL);
+                               }
+                       }
+                       break;
+               }
        if(ev->window == selmon->barwin && ev->x >= selmon->btx) {
                i = 0;
                x = selmon->btx;
        if(ev->window == selmon->barwin && ev->x >= selmon->btx) {
                i = 0;
                x = selmon->btx;
@@ -644,7 +659,7 @@ drawbar(Monitor *m) {
        }
        else
                x = dc.x;
        }
        else
                x = dc.x;
-       if(m == selmon) {
+       if(m == selmon) { /* status is only drawn on selected monitor */
                dc.w = TEXTW(stext);
                dc.x = m->ww - dc.w;
                if(dc.x < x) {
                dc.w = TEXTW(stext);
                dc.x = m->ww - dc.w;
                if(dc.x < x) {
@@ -652,20 +667,19 @@ drawbar(Monitor *m) {
                        dc.w = m->ww - x;
                }
                drawtext(stext, dc.norm, False);
                        dc.w = m->ww - x;
                }
                drawtext(stext, dc.norm, False);
-               if((dc.w = dc.x - x) > bh) {
-                       dc.x = x;
-                       if(selmon->sel) {
-                               drawtext(selmon->sel->name, dc.sel, False);
-                               drawsquare(selmon->sel->isfixed, selmon->sel->isfloating, False, dc.sel);
-                       }
-                       else
-                               drawtext(NULL, dc.norm, False);
-               }
        }
        else {
        }
        else {
+               dc.x = m->ww;
+       }
+       if((dc.w = dc.x - x) > bh) {
                dc.x = x;
                dc.x = x;
-               dc.w = m->ww - x;
-               drawtext(NULL, dc.norm, False);
+               if(m->sel) {
+                       col = m == selmon ? dc.sel : dc.norm;
+                       drawtext(m->sel->name, col, False);
+                       drawsquare(m->sel->isfixed, m->sel->isfloating, False, col);
+               }
+               else
+                       drawtext(NULL, dc.norm, False);
        }
        XCopyArea(dpy, dc.drawable, m->barwin, dc.gc, 0, 0, m->ww, bh, 0, 0);
        XSync(dpy, False);
        }
        XCopyArea(dpy, dc.drawable, m->barwin, dc.gc, 0, 0, m->ww, bh, 0, 0);
        XSync(dpy, False);
@@ -758,11 +772,11 @@ void
 focus(Client *c) {
        if(!c || !ISVISIBLE(c))
                for(c = selmon->stack; c && !ISVISIBLE(c); c = c->snext);
 focus(Client *c) {
        if(!c || !ISVISIBLE(c))
                for(c = selmon->stack; c && !ISVISIBLE(c); c = c->snext);
-       if(selmon->sel && selmon->sel != c) {
-               grabbuttons(selmon->sel, False);
-               XSetWindowBorder(dpy, selmon->sel->win, dc.norm[ColBorder]);
-       }
+       if(selmon->sel)
+               unfocus(selmon->sel);
        if(c) {
        if(c) {
+               if(c->mon != selmon)
+                       selmon = c->mon;
                if(c->isurgent)
                        clearurgent(c);
                detachstack(c);
                if(c->isurgent)
                        clearurgent(c);
                detachstack(c);
@@ -793,8 +807,13 @@ focusmon(const Arg *arg) {
 
        for(i = 0, m = mons; m; m = m->next, i++)
                if(i == arg->ui) {
 
        for(i = 0, m = mons; m; m = m->next, i++)
                if(i == arg->ui) {
-                       selmon = m;
-                       focus(NULL);
+                       if(m->stack)
+                               focus(m->stack);
+                       else {
+                               unfocus(selmon->stack);
+                               selmon = m;
+                               focus(NULL);
+                       }
                        drawbars();
                        break;
                }
                        drawbars();
                        break;
                }
@@ -829,12 +848,14 @@ focusstack(const Arg *arg) {
 
 Client *
 getclient(Window w) {
 
 Client *
 getclient(Window w) {
-       Client *c = NULL;
+       Client *c;
        Monitor *m;
 
        for(m = mons; m; m = m->next)
        Monitor *m;
 
        for(m = mons; m; m = m->next)
-               for(c = m->clients; c && c->win != w; c = c->next);
-       return c;
+               for(c = m->clients; c; c = c->next)
+                       if(c->win == w)
+                               return c;
+       return NULL;
 }
 
 unsigned long
 }
 
 unsigned long
@@ -1212,8 +1233,7 @@ resize(Client *c, int x, int y, int w, int h) {
                c->w = wc.width = w;
                c->h = wc.height = h;
                wc.border_width = c->bw;
                c->w = wc.width = w;
                c->h = wc.height = h;
                wc.border_width = c->bw;
-               XConfigureWindow(dpy, c->win,
-                               CWX|CWY|CWWidth|CWHeight|CWBorderWidth, &wc);
+               XConfigureWindow(dpy, c->win, CWX|CWY|CWWidth|CWHeight|CWBorderWidth, &wc);
                configure(c);
                XSync(dpy, False);
        }
                configure(c);
                XSync(dpy, False);
        }
@@ -1582,6 +1602,14 @@ toggleview(const Arg *arg) {
        }
 }
 
        }
 }
 
+void
+unfocus(Client *c) {
+       if(!c)
+               return;
+       grabbuttons(c, False);
+       XSetWindowBorder(dpy, c->win, dc.norm[ColBorder]);
+}
+
 void
 unmanage(Client *c) {
        XWindowChanges wc;
 void
 unmanage(Client *c) {
        XWindowChanges wc;
@@ -1649,9 +1677,11 @@ updatebarpos(Monitor *m) {
 
 void
 updategeom(void) {
 
 void
 updategeom(void) {
-       int i, n = 1;
+       int i, di, n = 1, x, y;
+       unsigned int dui;
        Client *c;
        Monitor *newmons = NULL, *m, *tm;
        Client *c;
        Monitor *newmons = NULL, *m, *tm;
+       Window dummy;
 
 #ifdef XINULATOR
        n = 2;
 
 #ifdef XINULATOR
        n = 2;
@@ -1720,42 +1750,27 @@ updategeom(void) {
                m->showbar = showbar;
                m->topbar = topbar;
                updatebarpos(m);
                m->showbar = showbar;
                m->topbar = topbar;
                updatebarpos(m);
-               /* reassign all clients with same screen number */
-               for(tm = mons; tm; tm = tm->next)
-                       if(tm->screen_number == m->screen_number) {
-                               m->clients = tm->clients;
-                               m->sel = m->stack = tm->stack;
-                               tm->clients = NULL;
-                               tm->stack = NULL;
-                               for(c = m->clients; c; c = c->next)
-                                       c->mon = m;
-                       }
        }
 
        /* reassign left over clients of disappeared monitors */
        }
 
        /* reassign left over clients of disappeared monitors */
-       for(tm = mons; tm; tm = tm->next) {
-               while((c = tm->clients)) {
-                       detach(c);
+       for(tm = mons; tm; tm = tm->next)
+               while(tm->clients) {
+                       c = tm->clients;
+                       tm->clients = c->next;
                        detachstack(c);
                        c->mon = newmons;
                        attach(c);
                        attachstack(c);
                }
                        detachstack(c);
                        c->mon = newmons;
                        attach(c);
                        attachstack(c);
                }
-       }
 
        /* select focused monitor */
 
        /* select focused monitor */
-       if(!selmon) {
-               selmon = newmons;
-               int di, x, y;
-               unsigned int dui;
-               Window dummy;
-               if(XQueryPointer(dpy, root, &dummy, &dummy, &x, &y, &di, &di, &dui)) 
-                       for(m = newmons; m; m = m->next)
-                               if(INRECT(x, y, m->wx, m->wy, m->ww, m->wh)) {
-                                       selmon = m;
-                                       break;
-                               }
-       }
+       selmon = newmons;
+       if(XQueryPointer(dpy, root, &dummy, &dummy, &x, &y, &di, &di, &dui)) 
+               for(m = newmons; m; m = m->next)
+                       if(INRECT(x, y, m->wx, m->wy, m->ww, m->wh)) {
+                               selmon = m;
+                               break;
+                       }
 
        /* final assignment of new monitors */
        cleanupmons();
 
        /* final assignment of new monitors */
        cleanupmons();