this variant is known to work, but focus() is ugly - we need in general a better...

[dwm.git] / event.c

diff --git a/event.c b/event.c
index a6100e6..2e021d5 100644 (file)
--- a/event.c
+++ b/event.c
@@ -1,4 +1,4 @@
-/* (C)opyright MMVII Anselm R. Garbe <garbeam at gmail dot com>
+/* (C)opyright MMVI-MMVII Anselm R. Garbe <garbeam at gmail dot com>
  * See LICENSE file for license details.
  */
 #include "dwm.h"
@@ -35,14 +35,16 @@ movemouse(Client *c) {
        c->ismax = False;
        XQueryPointer(dpy, root, &dummy, &dummy, &x1, &y1, &di, &di, &dui);
        for(;;) {
-               XMaskEvent(dpy, MOUSEMASK | ExposureMask, &ev);
+               XMaskEvent(dpy, MOUSEMASK | ExposureMask | SubstructureRedirectMask, &ev);
                switch (ev.type) {
                case ButtonRelease:
-                       resize(c, True, TopLeft);
+                       resize(c, True);
                        XUngrabPointer(dpy, CurrentTime);
                        return;
+               case ConfigureRequest:
                case Expose:
-                       handler[Expose](&ev);
+               case MapRequest:
+                       handler[ev.type](&ev);
                        break;
                case MotionNotify:
                        XSync(dpy, False);
@@ -50,13 +52,13 @@ movemouse(Client *c) {
                        c->y = ocy + (ev.xmotion.y - y1);
                        if(abs(wax + c->x) < SNAP)
                                c->x = wax;
-                       else if(abs((wax + waw) - (c->x + c->w)) < SNAP)
-                               c->x = wax + waw - c->w - 2 * BORDERPX;
+                       else if(abs((wax + waw) - (c->x + c->w + 2 * c->border)) < SNAP)
+                               c->x = wax + waw - c->w - 2 * c->border;
                        if(abs(way - c->y) < SNAP)
                                c->y = way;
-                       else if(abs((way + wah) - (c->y + c->h)) < SNAP)
-                               c->y = way + wah - c->h - 2 * BORDERPX;
-                       resize(c, False, TopLeft);
+                       else if(abs((way + wah) - (c->y + c->h + 2 * c->border)) < SNAP)
+                               c->y = way + wah - c->h - 2 * c->border;
+                       resize(c, False);
                        break;
                }
        }
@@ -66,7 +68,6 @@ static void
 resizemouse(Client *c) {
        int ocx, ocy;
        int nw, nh;
-       Corner sticky;
        XEvent ev;
 
        ocx = c->x;
@@ -75,30 +76,26 @@ resizemouse(Client *c) {
                        None, cursor[CurResize], CurrentTime) != GrabSuccess)
                return;
        c->ismax = False;
-       XWarpPointer(dpy, None, c->win, 0, 0, 0, 0, c->w, c->h);
+       XWarpPointer(dpy, None, c->win, 0, 0, 0, 0, c->w + c->border - 1, c->h + c->border - 1);
        for(;;) {
-               XMaskEvent(dpy, MOUSEMASK | ExposureMask, &ev);
+               XMaskEvent(dpy, MOUSEMASK | ExposureMask | SubstructureRedirectMask , &ev);
                switch(ev.type) {
                case ButtonRelease:
-                       resize(c, True, TopLeft);
+                       resize(c, True);
                        XUngrabPointer(dpy, CurrentTime);
                        return;
+               case ConfigureRequest:
                case Expose:
-                       handler[Expose](&ev);
+               case MapRequest:
+                       handler[ev.type](&ev);
                        break;
                case MotionNotify:
                        XSync(dpy, False);
-                       if((nw = abs(ocx - ev.xmotion.x)))
-                               c->w = nw;
-                       if((nh = abs(ocy - ev.xmotion.y)))
-                               c->h = nh;
-                       c->x = (ocx <= ev.xmotion.x) ? ocx : ocx - c->w;
-                       c->y = (ocy <= ev.xmotion.y) ? ocy : ocy - c->h;
-                       if(ocx <= ev.xmotion.x)
-                               sticky = (ocy <= ev.xmotion.y) ? TopLeft : BotLeft;
-                       else
-                               sticky = (ocy <= ev.xmotion.y) ? TopRight : BotRight;
-                       resize(c, True, sticky);
+                       nw = ev.xmotion.x - ocx - 2 * c->border + 1;
+                       c->w = nw > 0 ? nw : 1;
+                       nh = ev.xmotion.y - ocy - 2 * c->border + 1;
+                       c->h = nh > 0 ? nh : 1;
+                       resize(c, True);
                        break;
                }
        }
@@ -131,8 +128,20 @@ buttonpress(XEvent *e) {
                                return;
                        }
                }
-               if((ev->x < x + bmw) && (ev->button == Button1))
-                       togglemode(NULL);
+               if(ev->x < x + bmw)
+                       switch(ev->button) {
+                       case Button1:
+                               togglemode(NULL);
+                               break;
+                       case Button4:
+                               a.i = 1;
+                               incnmaster(&a);
+                               break;
+                       case Button5:
+                               a.i = -1;
+                               incnmaster(&a);
+                               break;
+                       }
        }
        else if((c = getclient(ev->window))) {
                focus(c);
@@ -182,9 +191,9 @@ configurerequest(XEvent *e) {
                        configure(c);
                XSync(dpy, False);
                if(c->isfloat) {
-                       resize(c, False, TopLeft);
+                       resize(c, False);
                        if(!isvisible(c))
-                               ban(c);
+                               XMoveWindow(dpy, c->win, c->x + 2 * sw, c->y);
                }
                else
                        arrange();
@@ -218,25 +227,22 @@ enternotify(XEvent *e) {
 
        if(ev->mode != NotifyNormal || ev->detail == NotifyInferior)
                return;
-       if(((c = getclient(ev->window)) || (c = getctitle(ev->window))) && isvisible(c))
+       if((c = getclient(ev->window)) && isvisible(c))
                focus(c);
        else if(ev->window == root) {
                issel = True;
-               XSetInputFocus(dpy, root, RevertToPointerRoot, CurrentTime);
-               drawall();
+               for(c = stack; c && !isvisible(c); c = c->snext);
+               focus(c);
        }
 }
 
 static void
 expose(XEvent *e) {
-       Client *c;
        XExposeEvent *ev = &e->xexpose;
 
        if(ev->count == 0) {
                if(barwin == ev->window)
                        drawstatus();
-               else if((c = getctitle(ev->window)))
-                       drawtitle(c);
        }
 }
 
@@ -264,7 +270,7 @@ leavenotify(XEvent *e) {
 
        if((ev->window == root) && !ev->same_screen) {
                issel = False;
-               drawall();
+               focus(sel);
        }
 }
 
@@ -319,8 +325,8 @@ propertynotify(XEvent *e) {
                }
                if(ev->atom == XA_WM_NAME || ev->atom == netatom[NetWMName]) {
                        updatetitle(c);
-                       resizetitle(c);
-                       drawtitle(c);
+                       if(c == sel)
+                               drawstatus();
                }
        }
 }