add to teh focus order before the call to change_state, since this can remove/add...

diff --git a/openbox/client.c b/openbox/client.c
index 94f571bb4f2540d5ebe87541af870c175647dc56..2e71f279667ddcaa262e935940803bee395e8632 100644 (file)
--- 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);