Don't kill keygrabs when focus moves. Applications should be better behaved by now, and GTK based apps seem to be at least. We can file bug reports with them if we need to still.
Dana Jansens danakj@orodu.net
2 files changed,
0 insertions(+),
12 deletions(-)
M
openbox/client.c
→
openbox/client.c
@@ -2621,10 +2621,6 @@ {
gboolean hide = FALSE; if (!client_should_show(self)) { - if (self == focus_client) { - event_cancel_all_key_grabs(); - } - /* We don't need to ignore enter events here. The window can hide/iconify in 3 different ways: 1 - through an x message. in this case we ignore all enter events@@ -3874,8 +3870,6 @@
/* if using focus_delay, stop the timer now so that focus doesn't go moving on us */ event_halt_focus_delay(); - - event_cancel_all_key_grabs(); xerror_set_ignore(TRUE); xerror_occured = FALSE;
M
openbox/focus.c
→
openbox/focus.c
@@ -83,10 +83,6 @@ /* uninstall the old colormap, and install the new one */
screen_install_colormap(focus_client, FALSE); screen_install_colormap(client, TRUE); - /* in the middle of cycling..? kill it. */ - focus_cycle_stop(focus_client); - focus_cycle_stop(client); - focus_client = client; if (client != NULL) {@@ -196,8 +192,6 @@ }
/* nothing is focused, update the colormap and _the root property_ */ focus_set_client(NULL); - - event_cancel_all_key_grabs(); /* when nothing will be focused, send focus to the backup target */ XSetInputFocus(ob_display, screen_support_win, RevertToPointerRoot,