don't click-to-place windows when they specify a position

diff --git a/src/Window.cc b/src/Window.cc
index 8df9ddfe733af2da1c99ded8fa80de0da9ef7e1a..cff95ec64e005e72ec7e68480e6edf0370d2a255 100644 (file)
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -216,7 +216,7 @@ OpenboxWindow::OpenboxWindow(Openbox &o, Window w, BScreen *s) : openbox(o) {
   }
   upsize();
 
-  Bool place_window = True;
+  place_window = true;
   if (openbox.isStartup() || flags.transient ||
       client.normal_hint_flags & (PPosition|USPosition)) {
     setGravityOffsets();
@@ -226,7 +226,7 @@ OpenboxWindow::OpenboxWindow(Openbox &o, Window w, BScreen *s) : openbox(o) {
         (signed) (frame.y + frame.y_border) >= 0 &&
         frame.x <= (signed) screen->size().w() &&
         frame.y <= (signed) screen->size().h()))
-      place_window = False;
+      place_window = false;
   }
 
   frame.window = createToplevelWindow(frame.x, frame.y, frame.width,
@@ -1467,8 +1467,8 @@ void OpenboxWindow::deiconify(Bool reassoc, Bool raise) {
 
   // if we're using the click to place placement type, then immediately
   // after the window is mapped, we need to start interactively moving it
-  if (!(flags.iconic || reassoc) &&
-      screen->placementPolicy() == BScreen::ClickMousePlacement) {
+  if (screen->placementPolicy() == BScreen::ClickMousePlacement &&
+      place_window && !(flags.iconic || reassoc)) {
     // if the last window wasn't placed yet, or we're just moving a window
     // already, finish off that move cleanly
     OpenboxWindow *w = openbox.getFocusedWindow();

diff --git a/src/Window.h b/src/Window.h
index faaf9a0031e16e89849ed88abca2c016a4b16529..66a3049ea40072699648dcacdee7e23219cd6d81 100644 (file)
--- a/src/Window.h
+++ b/src/Window.h
@@ -78,6 +78,7 @@ private:
 
   int window_number, workspace_number;
   unsigned long current_state;
+  bool place_window;         // place the window ourselves when mapped
 
   enum FocusMode { F_NoInput = 0, F_Passive,
                   F_LocallyActive, F_GloballyActive };