changed shortcuts as described on the mailinglist, added sanity checks for ratios...

[dwm.git] / layout.c

diff --git a/layout.c b/layout.c
index fa7b4eb..2016917 100644 (file)
--- a/layout.c
+++ b/layout.c
@@ -13,24 +13,6 @@ static double vratio = VRATIO;
 static unsigned int nlayouts = 0;
 static unsigned int nmaster = NMASTER;
 
 static unsigned int nlayouts = 0;
 static unsigned int nmaster = NMASTER;
 

-static void
-incratio(const char *arg, double *ratio, double def) {
-       double delta;
-
-       if(lt->arrange != tile)
-               return;
-       if(!arg)
-               *ratio = def;
-       else {
-               if(1 == sscanf(arg, "%lf", &delta)) {
-                       if(delta + (*ratio) < .1 || delta + (*ratio) > 1.9)
-                               return;
-                       *ratio += delta;
-               }
-       }
-       lt->arrange();
-}
-

 static double /* simple pow() */
 spow(double x, double y)
 {
 static double /* simple pow() */
 spow(double x, double y)
 {


@@ -43,21 +25,27 @@ spow(double x, double y)
 
 static void
 tile(void) {
 
 static void
 tile(void) {

-       double mscale = 0, tscale = 0, sum = 0;
+       Bool mmaxtile = False, smaxtile = False; /* fallback tiling */
+       double mscale = 0, sscale = 0, sum = 0;

        unsigned int i, n, nx, ny, nw, nh, mw, tw;
        Client *c;
 
        unsigned int i, n, nx, ny, nw, nh, mw, tw;
        Client *c;
 

+       /* preparation */

        for(n = 0, c = nexttiled(clients); c; c = nexttiled(c->next))
                n++;
        for(n = 0, c = nexttiled(clients); c; c = nexttiled(c->next))
                n++;

-
+       nx = wax;
+       ny = way;

        mw = (n <= nmaster) ? waw :  waw / (1 + hratio);
        tw = waw - mw;
        mw = (n <= nmaster) ? waw :  waw / (1 + hratio);
        tw = waw - mw;

-

        if(n > 0) {
        if(n > 0) {

-               if(n < nmaster) {
+               if(n <= nmaster) {

                        for(i = 0; i < n; i++)
                                sum += spow(vratio, i);
                        mscale = wah / sum;
                        for(i = 0; i < n; i++)
                                sum += spow(vratio, i);
                        mscale = wah / sum;

+                       if(vratio >= 1)
+                               mmaxtile = bh > (mscale * spow(vratio, 0));
+                       else
+                               mmaxtile = bh > (mscale * spow(vratio, n - 1));

                }
                else {
                        for(i = 0; i < nmaster; i++)
                }
                else {
                        for(i = 0; i < nmaster; i++)


@@ -65,11 +53,18 @@ tile(void) {
                        mscale = wah / sum;
                        for(sum = 0, i = 0; i < (n - nmaster); i++)
                                sum += spow(vratio, i);
                        mscale = wah / sum;
                        for(sum = 0, i = 0; i < (n - nmaster); i++)
                                sum += spow(vratio, i);

-                       tscale = wah / sum;
+                       sscale = wah / sum;
+                       if(vratio >= 1) {
+                               mmaxtile = bh > (mscale * spow(vratio, 0));
+                               smaxtile = bh > (sscale * spow(vratio, 0));
+                       }
+                       else {
+                               mmaxtile = bh > (mscale * spow(vratio, nmaster - 1));
+                               smaxtile = bh > (sscale * spow(vratio, n - nmaster - 1));
+                       }

                }
        }
                }
        }

-       nx = wax;
-       ny = way;
+       /* tiling */

        for(i = 0, c = clients; c; c = c->next)
                if(isvisible(c)) {
                        unban(c);
        for(i = 0, c = clients; c; c = c->next)
                if(isvisible(c)) {
                        unban(c);


@@ -78,25 +73,29 @@ tile(void) {
                        c->ismax = False;
                        if(i < nmaster) { /* master window */
                                nw = mw - 2 * c->border;
                        c->ismax = False;
                        if(i < nmaster) { /* master window */
                                nw = mw - 2 * c->border;

-                               if(i + 1 == n || i + 1 == nmaster)
-                                       nh = (way + wah) - ny - (2 * c->border);
+                               if(mmaxtile) {
+                                       ny = way;
+                                       nh = wah - 2 * c->border;
+                               }
+                               else if(i + 1 == (n < nmaster ? n : nmaster))
+                                       nh = (way + wah) - ny - 2 * c->border;

                                else
                                else

-                                       nh = (mscale * spow(vratio, i)) - (2 * c->border);
+                                       nh = (mscale * spow(vratio, i)) - 2 * c->border;

                        }
                        else { /* tile window */
                        }
                        else { /* tile window */

+                               nw = tw - 2 * c->border;

                                if(i == nmaster) {
                                        ny = way;
                                        nx = wax + mw;
                                }
                                if(i == nmaster) {
                                        ny = way;
                                        nx = wax + mw;
                                }

-                               nw = tw - 2 * c->border;
-                               if(i + 1 == n)
-                                       nh = (way + wah) - ny - (2 * c->border);
+                               if(smaxtile) {
+                                       ny = way;
+                                       nh = wah - 2 * c->border;
+                               }
+                               else if(i + 1 == n)
+                                       nh = (way + wah) - ny - 2 * c->border;

                                else
                                else

-                                       nh = (tscale * spow(vratio, i - nmaster)) - (2 * c->border);
-                       }
-                       if(nh < bh) {
-                               nh = bh;
-                               ny = way + wah - nh;
+                                       nh = (sscale * spow(vratio, i - nmaster)) - 2 * c->border;

                        }
                        resize(c, nx, ny, nw, nh, False);
                        ny += nh;
                        }
                        resize(c, nx, ny, nw, nh, False);
                        ny += nh;


@@ -110,6 +109,24 @@ tile(void) {
 
 LAYOUTS
 
 
 LAYOUTS
 

+static void
+incratio(const char *arg, double *ratio, double def) {
+       double delta;
+
+       if(lt->arrange != tile)
+               return;
+       if(!arg)
+               *ratio = def;
+       else {
+               if(1 == sscanf(arg, "%lf", &delta)) {
+                       if(delta + (*ratio) < .1 || delta + (*ratio) > 1.9)
+                               return;
+                       *ratio += delta;
+               }
+       }
+       lt->arrange();
+}
+

 /* extern */
 
 void
 /* extern */
 
 void