From: Dana Jansens <danakj@orodu.net>
Date: Mon, 4 Jun 2007 20:31:33 +0000 (+0000)
Subject: don't change the focus order when a client tries to focus but is not visible. client_... 
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=62d9a341381729a02ee174f25f197b22499e189f;p=chaz%2Fopenbox

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
---

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;