capitalization

[chaz/openbox] / src / screen.cc

diff --git a/src/screen.cc b/src/screen.cc
index d47a1540f83133130d2cf4e3e14c29e7c4914ba3..40f143a13539fe694610beb52733fa67c27bb734 100644 (file)
--- a/src/screen.cc
+++ b/src/screen.cc
@@ -134,7 +134,7 @@ Screen::Screen(int screen)
   openbox->registerHandler(_info->rootWindow(), this);
 
   // call the python Startup callbacks
-  EventData data(_number, 0, EventAction::Shutdown, 0);
+  EventData data(_number, 0, EventAction::Startup, 0);
   openbox->bindings()->fireEvent(&data);
 }
 
@@ -208,6 +208,7 @@ void Screen::manageExisting()
 
 void Screen::updateStrut()
 {
+  otk::Strut old = _strut;  
   _strut.left = _strut.right = _strut.top = _strut.bottom = 0;
 
   Client::List::iterator it, end = clients.end();
@@ -219,6 +220,12 @@ void Screen::updateStrut()
     _strut.bottom = std::max(_strut.bottom, s.bottom);
   }
   calcArea();
+
+  if (!(old == _strut)) {
+    // the strut has changed, adjust all the maximized windows
+    for (it = clients.begin(); it != end; ++it)
+      (*it)->remaximize();
+  }
 }
 
 
@@ -512,16 +519,15 @@ void Screen::manageWindow(Window window)
   // reparent the client to the frame
   client->frame->grabClient();
 
-  if (!(openbox->state() == Openbox::State_Starting ||
-        client->positionRequested())) {
+  if (openbox->state() != Openbox::State_Starting) {
     // position the window intelligenty .. hopefully :)
     // call the python PLACEWINDOW binding
     EventData data(_number, client, EventAction::PlaceWindow, 0);
     openbox->bindings()->fireEvent(&data);
   }
 
-  EventData data(_number, client, EventAction::DisplayingWindow, 0);
-  openbox->bindings()->fireEvent(&data);
+  EventData ddata(_number, client, EventAction::DisplayingWindow, 0);
+  openbox->bindings()->fireEvent(&ddata);
 
   // if on the current desktop.. (or all desktops)
   if (client->desktop() == _desktop ||
@@ -536,7 +542,7 @@ void Screen::manageWindow(Window window)
   // add to the screen's list
   clients.push_back(client);
   // once the client is in the list, update our strut to include the new
-  // client's
+  // client's (it is good that this happens after window placement!)
   updateStrut();
   // this puts into the stacking order, then raises it
   _stacking.push_back(client);
@@ -546,8 +552,8 @@ void Screen::manageWindow(Window window)
 
   openbox->bindings()->grabButtons(true, client);
 
-  EventData data(_number, client, EventAction::NewWindow, 0);
-  openbox->bindings()->fireEvent(&data);
+  EventData ndata(_number, client, EventAction::NewWindow, 0);
+  openbox->bindings()->fireEvent(&ndata);
 
 #ifdef DEBUG
   printf("Managed window 0x%lx frame 0x%lx\n",
@@ -608,6 +614,10 @@ void Screen::unmanageWindow(Client *client)
   // remove from the screen's list
   clients.remove(client);
 
+  // once the client is out of the list, update our strut to remove it's
+  // influence
+  updateStrut();
+
   // unfocus the client (calls the focus callbacks)
   client->unfocus();