M ChangeLog → ChangeLog

@@ -1,5 +1,8 @@ 
  (Format: Year/Month/Day)
 Changes for 1.0rc3:
+*07/02/17:
+   * Fixed some issues with transients of inactive tabs (Mark)
+     Window.cc
 *07/02/10:
    * MaxSize and MinSize hints weren't getting updated properly -- bug #1560803
      (Mark + thanks Jim Ramsay)

M src/Window.cc → src/Window.cc

@@ -1887,11 +1887,11 @@ m_client->transientFor()->transientList().push_back(m_client);
     }
     // raise this window and every transient in it with this one last
     if (client->fbwindow()) {
-        raiseFluxboxWindow(*client->fbwindow());
         // doing this on startup messes up the focus order
         if (!Fluxbox::instance()->isStartup())
             // activate the client so the transient won't get pushed back down
             client->fbwindow()->setCurrentClient(*client, false);
+        raiseFluxboxWindow(*client->fbwindow());
     }
 
 }
@@ -1917,15 +1917,9 @@ void FluxboxWindow::tempRaise() {
     if (isIconic())
         deiconify();
 
-    // get root window
-    WinClient *client = getRootTransientFor(m_client);
-
-    // if we don't have any root window use this as root
-    if (client == 0)
-        client = m_client;
-
-    if (client->fbwindow())
-        tempRaiseFluxboxWindow(*client->fbwindow());
+    // the root transient will get raised when we stop cycling
+    // raising it here causes problems when it isn't the active tab
+    tempRaiseFluxboxWindow(*this);
 }