don't change the focus order when a client tries to focus but is not visible. client_...

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

diff --git a/openbox/client.c b/openbox/client.c
index daa65379f3d50e4201a60c7ea0f91976c3ed7126..fa67e367f14aa419bf6cec47a55b9b5f28231a36 100644 (file)
--- 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;