fixed some strange behavior with tabbing a transient to its main window

2 files changed, 4 insertions, 1 deletions

diff --git a/src/Ewmh.cc b/src/Ewmh.cc
index 646fa0e..ce5f1a3 100644
--- a/src/Ewmh.cc
+++ b/src/Ewmh.cc
@@ -328,6 +328,8 @@ void Ewmh::setupFrame(FluxboxWindow &win) {
                 changeProperty(m_net_wm_window_type,
                                XA_ATOM, 32, PropModeReplace,
                                (unsigned char*)&m_net_wm_window_type_dialog, 1);
+            // and then we should treat it like a dialog
+            win.setTabable(false);
         }
     }
 
diff --git a/src/Window.cc b/src/Window.cc
index 36bd271..ccdb477 100644
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -1887,7 +1887,8 @@ void FluxboxWindow::raise() {
     // raise this window and every transient in it with this one last
     if (client->fbwindow()) {
         // doing this on startup messes up the focus order
-        if (!Fluxbox::instance()->isStartup())
+        if (!Fluxbox::instance()->isStartup() && client->fbwindow() != this &&
+                &client->fbwindow()->winClient() != client)
             // activate the client so the transient won't get pushed back down
             client->fbwindow()->setCurrentClient(*client, false);
         raiseFluxboxWindow(*client->fbwindow());