using a global stack for focus recovery on arrange() - seems to work great

[dwm.git] / view.c

diff --git a/view.c b/view.c
index c005a33..5cbab08 100644 (file)
--- a/view.c
+++ b/view.c
@@ -12,13 +12,14 @@ minclient()
 {
        Client *c, *min;
 
 {
        Client *c, *min;
 

+       if((clients && clients->isfloat) || arrange == dofloat)
+               return clients; /* don't touch floating order */

        for(min = c = clients; c; c = c->next)
                if(c->weight < min->weight)
                        min = c;
        return min;
 }
 
        for(min = c = clients; c; c = c->next)
                if(c->weight < min->weight)
                        min = c;
        return min;
 }
 

-

 static void
 reorder()
 {
 static void
 reorder()
 {


@@ -38,6 +39,13 @@ reorder()
        clients = newclients;
 }
 
        clients = newclients;
 }
 

+static Client *
+nexttiled(Client *c)
+{
+       for(c = getnext(c); c && c->isfloat; c = getnext(c->next));
+       return c;
+}
+

 /* extern */
 
 void (*arrange)(Arg *) = DEFMODE;
 /* extern */
 
 void (*arrange)(Arg *) = DEFMODE;


@@ -68,8 +76,10 @@ dofloat(Arg *arg)
                else
                        ban(c);
        }
                else
                        ban(c);
        }

-       if(!sel || !isvisible(sel))
-               focus(getnext(clients));
+       if(!sel || !isvisible(sel)) {
+               for(sel = stack; sel && !isvisible(sel); sel = sel->snext);
+               focus(sel);
+       }

        restack();
 }
 
        restack();
 }
 


@@ -130,8 +140,10 @@ dotile(Arg *arg)
                else
                        ban(c);
        }
                else
                        ban(c);
        }

-       if(!sel || !isvisible(sel))
-               focus(getnext(clients));
+       if(!sel || !isvisible(sel)) {
+               for(sel = stack; sel && !isvisible(sel); sel = sel->snext);
+               focus(sel);
+       }

        restack();
 }
 
        restack();
 }
 


@@ -208,52 +220,22 @@ resizecol(Arg *arg)
 void
 restack()
 {
 void
 restack()
 {

-       static unsigned int nwins = 0;
-       static Window *wins = NULL;
-       unsigned int f, fi, m, mi, n;

        Client *c;
        XEvent ev;
        Client *c;
        XEvent ev;

-
-       for(f = 0, m = 0, c = clients; c; c = c->next)
-               if(isvisible(c)) {
-                       if(c->isfloat || arrange == dofloat)
-                               f++;
-                       else
-                               m++;
-               }
-       if(!(n = 2 * (f + m))) {
+       
+       if(!sel) {

                drawstatus();
                return;
        }
                drawstatus();
                return;
        }

-       if(nwins < n) {
-               nwins = n;
-               wins = erealloc(wins, nwins * sizeof(Window));
-       }
-
-       fi = 0;
-       mi = 2 * f;
-       if(sel) {
-               if(sel->isfloat || arrange == dofloat) {
-                       wins[fi++] = sel->twin;
-                       wins[fi++] = sel->win;
-               }
-               else {
-                       wins[mi++] = sel->twin;
-                       wins[mi++] = sel->win;
-               }
+       if(sel->isfloat || arrange == dofloat) {
+               XRaiseWindow(dpy, sel->win);
+               XRaiseWindow(dpy, sel->twin);

        }
        }

-       for(c = clients; c; c = c->next)
-               if(isvisible(c) && c != sel) {
-                       if(c->isfloat || arrange == dofloat) {
-                               wins[fi++] = c->twin;
-                               wins[fi++] = c->win;
-                       }
-                       else {
-                               wins[mi++] = c->twin;
-                               wins[mi++] = c->win;
-                       }
+       if(arrange != dofloat)
+               for(c = nexttiled(clients); c; c = nexttiled(c->next)) {
+                       XLowerWindow(dpy, c->twin);
+                       XLowerWindow(dpy, c->win);

                }
                }

-       XRestackWindows(dpy, wins, n);

        drawall();
        XSync(dpy, False);
        while(XCheckMaskEvent(dpy, EnterWindowMask, &ev));
        drawall();
        XSync(dpy, False);
        while(XCheckMaskEvent(dpy, EnterWindowMask, &ev));


@@ -317,12 +299,13 @@ zoom(Arg *arg)
        if(!sel || sel->isfloat || n < 2 || (arrange != dotile) || maximized)
                return;
 
        if(!sel || sel->isfloat || n < 2 || (arrange != dotile) || maximized)
                return;
 

-       if((c = sel)  == getnext(clients))
-               if(!(c = getnext(c->next)))
+       if((c = sel) == nexttiled(clients))
+               if(!(c = nexttiled(c->next)))

                        return;
        detach(c);
                        return;
        detach(c);

+       if(clients)
+               clients->prev = c;

        c->next = clients;
        c->next = clients;

-       clients->prev = c;

        clients = c;
        focus(c);
        arrange(NULL);
        clients = c;
        focus(c);
        arrange(NULL);