all repos — openbox @ fa903969765a42df1b2d756ff7a183a2660c1457

openbox fork - make it a bit more like ryudo 

optimized the finding-the-slit-position code in positionWindow
Dana Jansens danakj@orodu.net
commit

fa903969765a42df1b2d756ff7a183a2660c1457

parent

83b39a9a3e366d9bc4baadb7c988b874cfe25a08

1 files changed, 9 insertions(+), 19 deletions(-)

jump to
M src/Workspace.cc → src/Workspace.cc 
@@ -549,38 +549,28 @@ if ((slit->direction() == Slit::Horizontal &&
        (slit->placement() == Slit::TopLeft ||
         slit->placement() == Slit::TopRight)) ||
       slit->placement() == Slit::TopCenter)
-    remove = 0;
+    // exclude top
+    space.setY(slit->area().h() + screen.getBorderWidth() * 2);
   else if ((slit->direction() == Slit::Vertical &&
             (slit->placement() == Slit::TopRight ||
              slit->placement() == Slit::BottomRight)) ||
            slit->placement() == Slit::CenterRight)
-    remove = 1;
+    // exclude right
+    space.setW(screen.size().w() -
+               (slit->area().w() + screen.getBorderWidth() * 2));
   else if ((slit->direction() == Slit::Horizontal &&
             (slit->placement() == Slit::BottomLeft ||
              slit->placement() == Slit::BottomRight)) ||
            slit->placement() == Slit::TopCenter)
-    remove = 2;
+    // exclude bottom
+    space.setH(screen.size().h() -
+               (slit->area().h() + screen.getBorderWidth() * 2));
   else// if ((slit->direction() == Slit::Vertical &&
       //      (slit->placement() == Slit::TopLeft ||
       //       slit->placement() == Slit::BottomLeft)) ||
       //     slit->placement() == Slit::CenterLeft)
-    remove = 3;
-  switch (remove) {
-  case 0: // top
-    space.setY(slit->area().h() + screen.getBorderWidth() * 2);
-    break;
-  case 1: // right
-    space.setW(screen.size().w() -
-               (slit->area().w() + screen.getBorderWidth() * 2));
-    break;
-  case 2: // bottom
-    space.setH(screen.size().h() -
-               (slit->area().h() + screen.getBorderWidth() * 2));
-    break;
-  case 3: // left
+    // exclude left
     space.setX(slit->area().w() + screen.getBorderWidth() * 2);
-    break;
-  }
 #endif
 
   Toolbar *toolbar = screen.getToolbar();