make sure we don't steal focus from unmanaged windows

1 files changed, 14 insertions, 2 deletions

diff --git a/src/fluxbox.cc b/src/fluxbox.cc
index e4c68bf..2393837 100644
--- a/src/fluxbox.cc
+++ b/src/fluxbox.cc
@@ -1719,8 +1719,20 @@ void Fluxbox::timed_reconfigure() {
 }
 
 void Fluxbox::revert_focus() {
-    if (m_revert_screen && !FocusControl::focusedWindow() &&
-        !FbTk::Menu::focused() && !m_showing_dialog)
+    if (!m_revert_screen || FocusControl::focusedWindow() ||
+            FbTk::Menu::focused() || m_showing_dialog)
+        return;
+
+    Window win;
+    int revert;
+    Display *disp = display();
+
+    XGetInputFocus(disp, &win, &revert);
+
+    // we only want to revert focus if it's left dangling, as some other
+    // application may have set the focus to an unmanaged window
+    if (win == None || win == PointerRoot ||
+            win == m_revert_screen->rootWindow().window())
         FocusControl::revertFocus(*m_revert_screen);
 }