X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;f=src%2FScreen.cc;h=2722fe4bd28310f1336109fda1ddec2569e830ad;hb=18f704edd0938355622049d853c1ce3cdfaee168;hp=5c1134ffc10d54834f50e0735d97dd51c5419693;hpb=3ee18e2f5f1477b70538fe9250ee498f6f583122;p=chaz%2Fopenbox

diff --git a/src/Screen.cc b/src/Screen.cc
index 5c1134ff..2722fe4b 100644
--- a/src/Screen.cc
+++ b/src/Screen.cc
@@ -1264,21 +1264,28 @@ void BScreen::changeWorkspaceID(unsigned int id) {
       XQueryPointer(blackbox->getXDisplay(), getRootWindow(), &r, &c,
                     &rx, &ry, &x, &y, &m) &&
       c != None) {
-    if ( (win = blackbox->searchWindow(c)) ) {
+    if ( (win = blackbox->searchWindow(c)) )
       f = win->setInputFocus();
-      blackbox->setFocusedWindow(win);
-    }        
   }
 
   // If that fails, and we're doing focus_last, try to focus the last window.
   if (! f && resource.focus_last &&
-      (win = current_workspace->getLastFocusedWindow())) {
+      (win = current_workspace->getLastFocusedWindow()))
     f = win->setInputFocus();
-    blackbox->setFocusedWindow(win);
-  }
 
-  // If that fails, then set focus to nothing.
-  if (! f) blackbox->setFocusedWindow((BlackboxWindow *) 0);
+  /*
+    if we found a focus target, then we set the focused window explicitly
+    because it is possible to switch off this workspace before the x server
+    generates the FocusIn event for the window. if that happens, openbox would
+    lose track of what window was the 'LastFocused' window on the workspace.
+
+    if we did not find a focus target, then set the current focused window to
+    nothing.
+  */
+  if (f)
+    blackbox->setFocusedWindow(win);
+  else
+    blackbox->setFocusedWindow((BlackboxWindow *) 0);
 
   updateNetizenCurrentWorkspace();
 }
@@ -1344,7 +1351,7 @@ void BScreen::updateStackingList(void) {
 void BScreen::addSystrayWindow(Window window) {
   XGrabServer(blackbox->getXDisplay());
   
-  XSelectInput(blackbox->getXDisplay(), window, SubstructureRedirectMask);
+  XSelectInput(blackbox->getXDisplay(), window, StructureNotifyMask);
   systrayWindowList.push_back(window);
   xatom->setValue(getRootWindow(), XAtom::kde_net_system_tray_windows,
                   XAtom::window,
@@ -2214,6 +2221,12 @@ void BScreen::shutdown(void) {
   while(! windowList.empty())
     unmanageWindow(windowList.front(), True);
 
+  while(! desktopWindowList.empty()) {
+    BlackboxWindow *win = blackbox->searchWindow(desktopWindowList.front());
+    assert(win);
+    unmanageWindow(win, True);
+  }
+
   slit->shutdown();
 }