add to teh focus order before the call to change_state, since this can remove/add to the focus order, and then we're in there twice

1 files changed, 4 insertions, 3 deletions

diff --git a/openbox/client.c b/openbox/client.c
index 94f571bb..2e71f279 100644
--- a/openbox/client.c
+++ b/openbox/client.c
@@ -269,6 +269,10 @@ void client_manage(Window window)
 
     sn_app_started(self->class);
 
+    /* update the focus lists, do this before the call to change_state or
+       it can end up in the list twice! */
+    focus_order_add_new(self);
+
     client_change_state(self);
 
     /* remove the client's border (and adjust re gravity) */
@@ -287,9 +291,6 @@ void client_manage(Window window)
 
     client_apply_startup_state(self);
 
-    /* update the focus lists */
-    focus_order_add_new(self);
-
     stacking_add(CLIENT_AS_WINDOW(self));
     client_restore_session_stacking(self);