]> Dogcows Code - chaz/openbox/blobdiff - src/Workspace.cc
Added a fallback for Best Fit to cascade. Also, moved cascade to its own
[chaz/openbox] / src / Workspace.cc
index 0e7edcab0aa506087b2e3f48f3708adc1cb5644a..df8133deee7ad5d64c701e74da0168e1baef8cd7 100644 (file)
@@ -41,6 +41,7 @@
 #include "Toolbar.h"
 #include "Window.h"
 #include "Workspace.h"
+
 #include "Windowmenu.h"
 #include "Geometry.h"
 #include "Util.h"
@@ -334,33 +335,33 @@ static rectList calcSpace(const OpenboxWindow &win, const rectList &spaces) {
   rectList result;
   rectList::const_iterator siter;
   for(siter=spaces.begin(); siter!=spaces.end(); ++siter) {
-    if(win.getArea().Intersect(*siter)) {
+    if(win.area().Intersect(*siter)) {
       //Check for space to the left of the window
-      if(win.getXFrame() > siter->x())
+      if(win.origin().x() > siter->x())
         result.push_back(Rect(siter->x(), siter->y(),
-                              win.getXFrame() - siter->x() - 1,
+                              win.origin().x() - siter->x() - 1,
                               siter->h()));
       //Check for space above the window
-      if(win.getYFrame() > siter->y())
+      if(win.origin().y() > siter->y())
         result.push_back(Rect(siter->x(), siter->y(),
                               siter->w(),
-                              win.getYFrame() - siter->y() - 1));
+                              win.origin().y() - siter->y() - 1));
       //Check for space to the right of the window
-      if((win.getXFrame()+win.getWidth()) <
+      if((win.origin().x()+win.size().w()) <
          (siter->x()+siter->w()))
-        result.push_back(Rect(win.getXFrame() + win.getWidth() + 1,
+        result.push_back(Rect(win.origin().x() + win.size().w() + 1,
                               siter->y(),
                               siter->x() + siter->w() -
-                              win.getXFrame() - win.getWidth() - 1,
+                              win.origin().x() - win.size().w() - 1,
                               siter->h()));
       //Check for space below the window
-      if((win.getYFrame()+win.getHeight()) <
+      if((win.origin().y()+win.size().h()) <
          (siter->y()+siter->h()))
         result.push_back(Rect(siter->x(),
-                              win.getYFrame() + win.getHeight() + 1,
+                              win.origin().y() + win.size().h() + 1,
                               siter->w(),
                               siter->y() + siter->h()-
-                              win.getYFrame() - win.getHeight() - 1));
+                              win.origin().y() - win.size().h() - 1));
 
     }
     else
@@ -386,17 +387,17 @@ Point *Workspace::bestFitPlacement(const Size &win_size, const Rect &space)
      spaces = calcSpace(*cur, spaces);
   
   //Find first space that fits the window
-  best = 0;
+  best = NULL;
   for (siter=spaces.begin(); siter!=spaces.end(); ++siter) {
     if ((siter->w() >= win_size.w()) &&
         (siter->h() >= win_size.h()))
       best = siter;
   }
 
-  if (best != 0)
+  if (best != NULL)
     return new Point(best->origin());
   else
-    return new Point(200, 0);
+    return NULL; //fall back to cascade
 }
 
 inline Point *Workspace::rowSmartPlacement(const Size &win_size,
@@ -412,48 +413,46 @@ inline Point *Workspace::rowSmartPlacement(const Size &win_size,
   LinkedListIterator<OpenboxWindow> it(windowList);
 
   test_y = (screen.getColPlacementDirection() == BScreen::TopBottom) ?
-    start_pos : screen.getHeight() - win_size.h() - start_pos;
+    start_pos : screen.size().h() - win_size.h() - start_pos;
 
   while(!placed &&
         ((screen.getColPlacementDirection() == BScreen::BottomTop) ?
          test_y > 0 : test_y + win_size.h() < (signed) space.h())) {
-     test_x = (screen.getRowPlacementDirection() == BScreen::LeftRight) ?
-              start_pos : space.w() - win_size.w() - start_pos;
-     while (!placed &&
-            ((screen.getRowPlacementDirection() == BScreen::RightLeft) ?
-             test_x > 0 : test_x + win_size.w() < (signed) space.w())) {
-        placed = true;
-    
-        it.reset();
-        for (OpenboxWindow *curr = it.current(); placed && curr;
-             it++, curr = it.current()) {
-           int curr_w = curr->getWidth() + (screen.getBorderWidth() * 4);
-           int curr_h =
-             ((curr->isShaded()) ? curr->getTitleHeight() : curr->getHeight()) +
-             (screen.getBorderWidth() * 4);
-   
-           if (curr->getXFrame() < test_x + win_size.w() &&
-               curr->getXFrame() + curr_w > test_x &&
-               curr->getYFrame() < test_y + win_size.h() &&
-               curr->getYFrame() + curr_h > test_y) {
-              placed = false;
-           }
+    test_x = (screen.getRowPlacementDirection() == BScreen::LeftRight) ?
+      start_pos : space.w() - win_size.w() - start_pos;
+    while (!placed &&
+           ((screen.getRowPlacementDirection() == BScreen::RightLeft) ?
+            test_x > 0 : test_x + win_size.w() < (signed) space.w())) {
+      placed = true;
+
+      it.reset();
+      for (OpenboxWindow *curr = it.current(); placed && curr;
+           it++, curr = it.current()) {
+        int curr_w = curr->size().w() + (screen.getBorderWidth() * 4);
+        int curr_h = curr->size().h() + (screen.getBorderWidth() * 4);
+
+        if (curr->origin().x() < test_x + win_size.w() &&
+            curr->origin().x() + curr_w > test_x &&
+            curr->origin().y() < test_y + win_size.h() &&
+            curr->origin().y() + curr_h > test_y) {
+          placed = false;
         }
-        // Removed code for checking toolbar and slit
-        // The space passed in should not include either
-        
-        if (placed) {
-           place_x = test_x;
-           place_y = test_y;
-           break;
-        }   
-        
-        test_x += (change_x * delta_x);
-     }
-
-     test_y += (change_y * delta_y);
+      }
+
+      // Removed code for checking toolbar and slit
+      // The space passed in should not include either
+
+      if (placed) {
+        place_x = test_x;
+        place_y = test_y;
+
+        break;
+      }   
+
+      test_x += (change_x * delta_x);
+    }
+
+    test_y += (change_y * delta_y);
   }
   return new Point(place_x, place_y);
 }
@@ -463,14 +462,14 @@ void Workspace::placeWindow(OpenboxWindow *win) {
 
   Bool placed = False;
 
-  const int win_w = win->getWidth() + (screen.getBorderWidth() * 4),
-    win_h = win->getHeight() + (screen.getBorderWidth() * 4),
+  const int win_w = win->size().w() + (screen.getBorderWidth() * 4),
+    win_h = win->size().h() + (screen.getBorderWidth() * 4),
 #ifdef    SLIT
-    slit_x = screen.getSlit()->getX() - screen.getBorderWidth(),
-    slit_y = screen.getSlit()->getY() - screen.getBorderWidth(),
-    slit_w = screen.getSlit()->getWidth() +
+    slit_x = screen.getSlit()->area().x() - screen.getBorderWidth(),
+    slit_y = screen.getSlit()->area().y() - screen.getBorderWidth(),
+    slit_w = screen.getSlit()->area().w() +
       (screen.getBorderWidth() * 4),
-    slit_h = screen.getSlit()->getHeight() +
+    slit_h = screen.getSlit()->area().h() +
       (screen.getBorderWidth() * 4),
 #endif // SLIT
     toolbar_x = screen.getToolbar()->getX() - screen.getBorderWidth(),
@@ -490,8 +489,8 @@ void Workspace::placeWindow(OpenboxWindow *win) {
   LinkedListIterator<OpenboxWindow> it(windowList);
 
   Rect space(0, 0,
-             screen.getWidth(),
-             screen.getHeight()
+             screen.size().w(),
+             screen.size().h()
             );
   Size window_size(win_w, win_h);
 
@@ -503,7 +502,9 @@ void Workspace::placeWindow(OpenboxWindow *win) {
       place_y=spot->y();
       delete spot;
       placed=true;
-    }
+    }else
+      placed=false;
+      
     break;
   }
   case BScreen::RowSmartPlacement: {
@@ -519,17 +520,17 @@ void Workspace::placeWindow(OpenboxWindow *win) {
 
   case BScreen::ColSmartPlacement: {
     test_x = (screen.getRowPlacementDirection() == BScreen::LeftRight) ?
-      start_pos : screen.getWidth() - win_w - start_pos;
+      start_pos : screen.size().w() - win_w - start_pos;
 
     while (!placed &&
           ((screen.getRowPlacementDirection() == BScreen::RightLeft) ?
-           test_x > 0 : test_x + win_w < (signed) screen.getWidth())) {
+           test_x > 0 : test_x + win_w < (signed) screen.size().w())) {
       test_y = (screen.getColPlacementDirection() == BScreen::TopBottom) ?
-       start_pos : screen.getHeight() - win_h - start_pos;
+       start_pos : screen.size().h() - win_h - start_pos;
       
       while (!placed &&
             ((screen.getColPlacementDirection() == BScreen::BottomTop) ?
-             test_y > 0 : test_y + win_h < (signed) screen.getHeight())) {
+             test_y > 0 : test_y + win_h < (signed) screen.size().h())) {
         placed = True;
 
         it.reset();
@@ -537,15 +538,15 @@ void Workspace::placeWindow(OpenboxWindow *win) {
             it++, curr = it.current()) {
           if (curr->isMaximizedFull()) // fully maximized, ignore it
             continue;
-          int curr_w = curr->getWidth() + (screen.getBorderWidth() * 4);
+          int curr_w = curr->size().w() + (screen.getBorderWidth() * 4);
           int curr_h =
-            ((curr->isShaded()) ? curr->getTitleHeight() : curr->getHeight()) +
+            ((curr->isShaded()) ? curr->getTitleHeight() : curr->size().h()) +
             (screen.getBorderWidth() * 4);
 
-          if (curr->getXFrame() < test_x + win_w &&
-              curr->getXFrame() + curr_w > test_x &&
-              curr->getYFrame() < test_y + win_h &&
-              curr->getYFrame() + curr_h > test_y) {
+          if (curr->origin().x() < test_x + win_w &&
+              curr->origin().x() + curr_w > test_x &&
+              curr->origin().y() < test_y + win_h &&
+              curr->origin().y() + curr_h > test_y) {
             placed = False;
          }
         }
@@ -583,21 +584,27 @@ void Workspace::placeWindow(OpenboxWindow *win) {
   } // switch
 
   if (! placed) {
-    if (((unsigned) cascade_x > (screen.getWidth() / 2)) ||
-       ((unsigned) cascade_y > (screen.getHeight() / 2)))
-      cascade_x = cascade_y = 32;
-
-    place_x = cascade_x;
-    place_y = cascade_y;
-
-    cascade_x += win->getTitleHeight();
-    cascade_y += win->getTitleHeight();
+    Point *p = cascade(win);
+    place_x=p->x();
+    place_y=p->y();
+    delete p;
   }
   
-  if (place_x + win_w > (signed) screen.getWidth())
-    place_x = (((signed) screen.getWidth()) - win_w) / 2;
-  if (place_y + win_h > (signed) screen.getHeight())
-    place_y = (((signed) screen.getHeight()) - win_h) / 2;
+  if (place_x + win_w > (signed) screen.size().w())
+    place_x = (((signed) screen.size().w()) - win_w) / 2;
+  if (place_y + win_h > (signed) screen.size().h())
+    place_y = (((signed) screen.size().h()) - win_h) / 2;
+
+  win->configure(place_x, place_y, win->size().w(), win->size().h());
+}
+
+Point *Workspace::cascade(const OpenboxWindow *const win){
+  if (((unsigned) cascade_x > (screen.size().w() / 2)) ||
+      ((unsigned) cascade_y > (screen.size().h() / 2)))
+    cascade_x = cascade_y = 32;
+
+  cascade_x += win->getTitleHeight();
+  cascade_y += win->getTitleHeight();
 
-  win->configure(place_x, place_y, win->getWidth(), win->getHeight());
+  return new Point(cascade_x, cascade_y);
 }
This page took 0.027675 seconds and 4 git commands to generate.