JasonWoof Got questions, comments, patches, etc.? Contact Jason Woofenden
using mon in function names for consistency reasons
[dwm.git] / dwm.c
diff --git a/dwm.c b/dwm.c
index dd6701b..5934a6c 100644 (file)
--- a/dwm.c
+++ b/dwm.c
@@ -166,7 +166,7 @@ static void detach(Client *c);
 static void detachstack(Client *c);
 static void die(const char *errstr, ...);
 static void drawbar(Monitor *m);
-static void drawbars();
+static void drawbars(void);
 static void drawsquare(Bool filled, Bool empty, Bool invert, unsigned long col[ColLast]);
 static void drawtext(const char *text, unsigned long col[ColLast], Bool invert);
 static void enternotify(XEvent *e);
@@ -176,6 +176,9 @@ static void focusin(XEvent *e);
 static void focusstack(const Arg *arg);
 static Client *getclient(Window w);
 static unsigned long getcolor(const char *colstr);
+static Monitor *getmon(Window w);
+static Monitor *getmonxy(int x, int y);
+static Bool getrootpointer(int *x, int *y);
 static long getstate(Window w);
 static Bool gettextprop(Window w, Atom atom, char *text, unsigned int size);
 static void grabbuttons(Client *c, Bool focused);
@@ -197,6 +200,7 @@ static void resizemouse(const Arg *arg);
 static void restack(Monitor *m);
 static void run(void);
 static void scan(void);
+static void sendmon(Client *c, Monitor *m);
 static void setclientstate(Client *c, long state);
 static void setlayout(const Arg *arg);
 static void setmfact(const Arg *arg);
@@ -211,6 +215,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 unfocus(Client *c);
 static void unmanage(Client *c);
 static void unmapnotify(XEvent *e);
 static void updategeom(void);
@@ -391,9 +396,16 @@ buttonpress(XEvent *e) {
        unsigned int i, x, click;
        Arg arg = {0};
        Client *c;
+       Monitor *m;
        XButtonPressedEvent *ev = &e->xbutton;
 
        click = ClkRootWin;
+       /* focus monitor if necessary */
+       if((m = getmon(ev->window)) && m != selmon) {
+               unfocus(selmon->sel);
+               selmon = m;
+               focus(NULL);
+       }
        if(ev->window == selmon->barwin && ev->x >= selmon->btx) {
                i = 0;
                x = selmon->btx;
@@ -644,7 +656,7 @@ drawbar(Monitor *m) {
        }
        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) {
@@ -652,27 +664,26 @@ drawbar(Monitor *m) {
                        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 {
+               dc.x = m->ww;
+       }
+       if((dc.w = dc.x - x) > bh) {
                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);
 }
 
 void
-drawbars() {
+drawbars(void) {
        Monitor *m;
 
        for(m = mons; m; m = m->next)
@@ -731,10 +742,15 @@ drawtext(const char *text, unsigned long col[ColLast], Bool invert) {
 void
 enternotify(XEvent *e) {
        Client *c;
+       Monitor *m;
        XCrossingEvent *ev = &e->xcrossing;
 
        if((ev->mode != NotifyNormal || ev->detail == NotifyInferior) && ev->window != root)
                return;
+       if((m = getmon(ev->window)) && m != selmon) {
+               unfocus(selmon->sel);
+               selmon = m;
+       }
        if((c = getclient(ev->window)))
                focus(c);
        else
@@ -746,23 +762,19 @@ expose(XEvent *e) {
        Monitor *m;
        XExposeEvent *ev = &e->xexpose;
 
-       if(ev->count == 0)
-               for(m = mons; m; m = m->next)
-                       if(ev->window == m->barwin) {
-                               drawbar(m);
-                               break;
-                       }
+       if(ev->count == 0 && (m = getmon(ev->window)))
+               drawbar(m);
 }
 
 void
 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->mon != selmon)
+                       selmon = c->mon;
                if(c->isurgent)
                        clearurgent(c);
                detachstack(c);
@@ -793,9 +805,11 @@ focusmon(const Arg *arg) {
 
        for(i = 0, m = mons; m; m = m->next, i++)
                if(i == arg->ui) {
+                       if(m == selmon)
+                               return;
+                       unfocus(selmon->sel);
                        selmon = m;
                        focus(NULL);
-                       drawbars();
                        break;
                }
 }
@@ -849,6 +863,40 @@ getcolor(const char *colstr) {
        return color.pixel;
 }
 
+Monitor *
+getmon(Window w) {
+       int x, y;
+       Client *c;
+       Monitor *m;
+
+       if(w == root && getrootpointer(&x, &y))
+               return getmonxy(x, y);
+       for(m = mons; m; m = m->next)
+               if(w == m->barwin)
+                       return m;
+       if((c = getclient(w)))
+               return c->mon;
+       return NULL;
+}
+
+Monitor *
+getmonxy(int x, int y) {
+       Monitor *m;
+
+       for(m = mons; m; m = m->next)
+               if(INRECT(x, y, m->wx, m->wy, m->ww, m->wh))
+                       return m;
+       return NULL;
+}
+
+Bool
+getrootpointer(int *x, int *y) {
+       int di;
+       unsigned int dui;
+       Window dummy;
+       return XQueryPointer(dpy, root, &dummy, &dummy, x, y, &di, &di, &dui);
+}
+
 long
 getstate(Window w) {
        int format, status;
@@ -1110,10 +1158,9 @@ monocle(Monitor *m) {
 
 void
 movemouse(const Arg *arg) {
-       int x, y, ocx, ocy, di, nx, ny;
-       unsigned int dui;
+       int x, y, ocx, ocy, nx, ny;
        Client *c;
-       Window dummy;
+       Monitor *m;
        XEvent ev;
 
        if(!(c = selmon->sel))
@@ -1124,7 +1171,8 @@ movemouse(const Arg *arg) {
        if(XGrabPointer(dpy, root, False, MOUSEMASK, GrabModeAsync, GrabModeAsync,
        None, cursor[CurMove], CurrentTime) != GrabSuccess)
                return;
-       XQueryPointer(dpy, root, &dummy, &dummy, &x, &y, &di, &di, &dui);
+       if(!getrootpointer(&x, &y))
+               return;
        do {
                XMaskEvent(dpy, MOUSEMASK|ExposureMask|SubstructureRedirectMask, &ev);
                switch (ev.type) {
@@ -1157,6 +1205,8 @@ movemouse(const Arg *arg) {
        }
        while(ev.type != ButtonRelease);
        XUngrabPointer(dpy, CurrentTime);
+       if((m = getmonxy(c->x + c->w / 2, c->y + c->h / 2)) != selmon)
+               sendmon(c, m);
 }
 
 Client *
@@ -1214,8 +1264,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;
-               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);
        }
@@ -1226,6 +1275,7 @@ resizemouse(const Arg *arg) {
        int ocx, ocy;
        int nw, nh;
        Client *c;
+       Monitor *m;
        XEvent ev;
 
        if(!(c = selmon->sel))
@@ -1264,6 +1314,8 @@ resizemouse(const Arg *arg) {
        XWarpPointer(dpy, None, c->win, 0, 0, 0, 0, c->w + c->bw - 1, c->h + c->bw - 1);
        XUngrabPointer(dpy, CurrentTime);
        while(XCheckMaskEvent(dpy, EnterWindowMask, &ev));
+       if((m = getmonxy(c->x + c->w / 2, c->y + c->h / 2)) != selmon)
+               sendmon(c, m);
 }
 
 void
@@ -1329,6 +1381,20 @@ scan(void) {
 }
 
 void
+sendmon(Client *c, Monitor *m) {
+       if(c->mon == m)
+               return;
+       detach(c);
+       detachstack(c);
+       c->mon = m;
+       c->tags = m->tagset[m->seltags]; /* assign tags of target monitor */
+       attach(c);
+       attachstack(c);
+       focus(NULL);
+       arrange();
+}
+
+void
 setclientstate(Client *c, long state) {
        long data[] = {state, None};
 
@@ -1484,14 +1550,7 @@ tagmon(const Arg *arg) {
                return;
        for(i = 0, m = mons; m; m = m->next, i++)
                if(i == arg->ui) {
-                       detach(c);
-                       detachstack(c);
-                       c->mon = m;
-                       attach(c);
-                       attachstack(c);
-                       selmon->sel = selmon->stack;
-                       m->sel = c;
-                       arrange();
+                       sendmon(c, m);
                        break;
                }
 }
@@ -1585,6 +1644,15 @@ toggleview(const Arg *arg) {
 }
 
 void
+unfocus(Client *c) {
+       if(!c)
+               return;
+       grabbuttons(c, False);
+       XSetWindowBorder(dpy, c->win, dc.norm[ColBorder]);
+       XSetInputFocus(dpy, root, RevertToPointerRoot, CurrentTime);
+}
+
+void
 unmanage(Client *c) {
        XWindowChanges wc;
 
@@ -1596,7 +1664,10 @@ unmanage(Client *c) {
        detach(c);
        detachstack(c);
        if(c->mon->sel == c) {
-               c->mon->sel = c->mon->stack;
+               /* TODO: consider separate the next code into a function or into detachstack? */
+               Client *tc;
+               for(tc = c->mon->stack; tc && !ISVISIBLE(tc); tc = tc->snext);
+               c->mon->sel = tc;
                focus(NULL);
        }
        XUngrabButton(dpy, AnyButton, AnyModifier, c->win);
@@ -1651,11 +1722,9 @@ updatebarpos(Monitor *m) {
 
 void
 updategeom(void) {
-       int i, di, n = 1, x, y;
-       unsigned int dui;
+       int i, n = 1;
        Client *c;
        Monitor *newmons = NULL, *m, *tm;
-       Window dummy;
 
 #ifdef XINULATOR
        n = 2;
@@ -1727,7 +1796,7 @@ updategeom(void) {
        }
 
        /* reassign left over clients of disappeared monitors */
-       for(tm = mons; tm; tm = tm->next) {
+       for(tm = mons; tm; tm = tm->next)
                while(tm->clients) {
                        c = tm->clients;
                        tm->clients = c->next;
@@ -1736,20 +1805,11 @@ updategeom(void) {
                        attach(c);
                        attachstack(c);
                }
-       }
 
        /* select focused monitor */
-       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();
        mons = newmons;
+       selmon = getmon(root);
 }
 
 void
@@ -1824,7 +1884,7 @@ updatetitle(Client *c) {
 }
 
 void
-updatestatus() {
+updatestatus(void) {
        if(!gettextprop(root, XA_WM_NAME, stext, sizeof(stext)))
                strcpy(stext, "dwm-"VERSION);
        drawbar(selmon);