all repos — openbox @ e7e2a69fd0ed5073a57ce2974892ad5cf97f7faa

openbox fork - make it a bit more like ryudo

handle multiple maprequests graciously
Dana Jansens danakj@orodu.net
commit

e7e2a69fd0ed5073a57ce2974892ad5cf97f7faa

parent

08a70966ee8fdec4204ab1ca9aed1371b1c828d0

1 files changed, 8 insertions(+), 9 deletions(-)

jump to
M openbox/event.copenbox/event.c

@@ -122,6 +122,9 @@ Client *client;

/* pick a window */ switch (e->type) { + case MapRequest: + window = e->xmap.window; + break; case UnmapNotify: window = e->xunmap.window; break;

@@ -248,12 +251,15 @@ break;

} client = g_hash_table_lookup(client_map, (gpointer)window); + g_message("EVENT: 0x%lx -> 0x%lx", window, client); /* deal with it in the kernel */ - if (client) { + if (client) event_handle_client(client, e); - } else if (window == ob_root) + else if (window == ob_root) event_handle_root(e); + else if (e->type == MapRequest) + client_manage(window); else if (e->type == ConfigureRequest) { /* unhandled configure requests must be used to configure the window directly */

@@ -286,10 +292,6 @@ {

Atom msgtype; switch(e->type) { - case MapRequest: - g_message("MapRequest on root"); - client_manage(e->xmap.window); - break; case ClientMessage: if (e->xclient.format != 32) break;

@@ -436,9 +438,6 @@

client_unmanage(client); break; case MapRequest: - /* we shouldn't be able to get this unless we're iconic */ - g_assert(client->iconic); - if (screen_showing_desktop) screen_show_desktop(FALSE); client_iconify(client, FALSE, TRUE);