all repos — openbox @ bb9376337331e4b1f4136fba0a5a0cb90ae2d3c6

openbox fork - make it a bit more like ryudo

grab bindings on each client's frame (and the focus_backup) instead of on the root window
Dana Jansens danakj@orodu.net
commit

bb9376337331e4b1f4136fba0a5a0cb90ae2d3c6

parent

8a1fa6d3ac47eba3e781463aec131cf401a29ec2

1 files changed, 34 insertions(+), 13 deletions(-)

jump to
M plugins/keyboard/keyboard.cplugins/keyboard/keyboard.c

@@ -1,4 +1,5 @@

#include "kernel/focus.h" +#include "kernel/frame.h" #include "kernel/dispatch.h" #include "kernel/openbox.h" #include "kernel/event.h"

@@ -80,19 +81,30 @@ static KeyBindingTree *curpos;

static guint reset_key, reset_state, button_return, button_escape; static Timer *chain_timer; -static void grab_keys() +static void grab_for_window(Window win, gboolean grab) { KeyBindingTree *p; - ungrab_all_keys(); + ungrab_all_keys(win); - p = curpos ? curpos->first_child : firstnode; - while (p) { - grab_key(p->key, p->state, GrabModeAsync); - p = p->next_sibling; + if (grab) { + p = curpos ? curpos->first_child : firstnode; + while (p) { + grab_key(p->key, p->state, win, GrabModeAsync); + p = p->next_sibling; + } + if (curpos) + grab_key(reset_key, reset_state, win, GrabModeAsync); } - if (curpos) - grab_key(reset_key, reset_state, GrabModeAsync); +} + +static void grab_keys(gboolean grab) +{ + GList *it; + + grab_for_window(focus_backup, grab); + for (it = client_list; it; it = g_list_next(it)) + grab_for_window(((Client*)it->data)->frame->window, grab); } static void reset_chains()

@@ -103,7 +115,7 @@ chain_timer = NULL;

} if (curpos) { curpos = NULL; - grab_keys(); + grab_keys(TRUE); } }

@@ -150,6 +162,14 @@ static void event(ObEvent *e, void *foo)

{ static KeyBindingTree *grabbed_key = NULL; + if (e->type == Event_Client_Mapped) { + grab_for_window(e->data.c.client->window, TRUE); + return; + } else if (e->type == Event_Client_Destroy) { + grab_for_window(e->data.c.client->window, FALSE); + return; + } + if (grabbed_key) { gboolean done = FALSE;

@@ -204,7 +224,7 @@ /* 5 second timeout for chains */

chain_timer = timer_start(5000*1000, chain_timeout, NULL); curpos = p; - grab_keys(); + grab_keys(TRUE); } else { GSList *it; for (it = p->actions; it; it = it->next) {

@@ -244,14 +264,15 @@

curpos = NULL; chain_timer = NULL; - dispatch_register(Event_X_KeyPress | Event_X_KeyRelease, + dispatch_register(Event_Client_Mapped | Event_Client_Destroy | + Event_X_KeyPress | Event_X_KeyRelease, (EventHandler)event, NULL); translate_key("C-g", &reset_state, &reset_key); translate_key("Escape", &i, &button_escape); translate_key("Return", &i, &button_return); - grab_keys(); + grab_keys(TRUE); } void plugin_shutdown()

@@ -260,6 +281,6 @@ dispatch_register(0, (EventHandler)event, NULL);

tree_destroy(firstnode); firstnode = NULL; - ungrab_all_keys(); + grab_keys(FALSE); }