don't change the focus order when a client tries to focus but is not visible. client_activate will -make- it visible anyways.

fixes focus fallback's loop totally breaking (stopping on 32bit and looping forever on 64bit) when it encountered a non-visible window

1 files changed, 0 insertions, 4 deletions

diff --git a/openbox/client.c b/openbox/client.c
index daa65379..fa67e367 100644
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -3519,10 +3519,6 @@ gboolean client_focus(ObClient *self)
     self = client_focus_target(self);
 
     if (!client_can_focus(self)) {
-        if (!self->frame->visible) {
-            /* update the focus lists */
-            focus_order_to_top(self);
-        }
         ob_debug_type(OB_DEBUG_FOCUS,
                       "Client %s can't be focused\n", self->title);
         return FALSE;