M openbox/action.c → openbox/action.c

@@ -1111,7 +1111,6 @@ gboolean cancel, gboolean done)
 {
     GSList *it;
     ObAction *a;
-    gboolean ungrabbed = FALSE;
 
     if (!acts)
         return;
@@ -1138,33 +1137,6 @@ a->data.inter.final = done;
                 if (!(cancel || done))
                     if (!keyboard_interactive_grab(state, a->data.any.c, a))
                         continue;
-            }
-            else if (!ungrabbed) {
-                /* Ungrab the keyboard before running the action if it's not
-                   interactive.
-
-                   If there is an interactive action going on, then cancel it
-                   to release the keyboard. If not, then call
-                   XUngrabKeyboard().
-
-                   We call XUngrabKeyboard() because a key press causes a
-                   passive grab on the keyboard, and so if program we are
-                   executing wants to grab the keyboard, it will fail if the
-                   button is still held down (which is likely).
-
-                   Use the X function not out own, because we're not
-                   considering a grab to be in place at all so our function
-                   won't try ungrab anything.
-                */
-                if (keyboard_interactively_grabbed())
-                    keyboard_interactive_cancel();
-                else
-                    XUngrabKeyboard(ob_display, time);
-
-                /* We don't the same with XUngrabPointer, even though it can
-                   cause the same problem. But then Press bindings cause
-                   Drag bindings to break.
-                XUngrabPointer(ob_display, time);*/
             }
 
             /* XXX UGLY HACK race with motion event starting a move and the
@@ -1240,6 +1212,12 @@ gchar *cmd, **argv = 0;
     if (data->execute.path) {
         cmd = g_filename_from_utf8(data->execute.path, -1, NULL, NULL, NULL);
         if (cmd) {
+            /* If there is an interactive action going on, then cancel it
+               to release the keyboard, so that the run application
+               can grab the keyboard if it wants to. */
+            if (keyboard_interactively_grabbed())
+                keyboard_interactive_cancel();
+
             if (!g_shell_parse_argv (cmd, NULL, &argv, &e)) {
                 g_message(_("Failed to execute '%s': %s"),
                           cmd, e->message);

M openbox/keyboard.c → openbox/keyboard.c

@@ -311,6 +311,14 @@ set_curpos(p);
             else {
                 keyboard_reset_chains(0);
 
+                /* If we don't have the keyboard grabbed, then ungrab it with
+                   XUngrabKeyboard, so that there is not a passive grab left
+                   on from the KeyPress. If the grab is left on, and focus
+                   moves during that time, it will be NotifyWhileGrabbed, and
+                   applications like to ignore those! */
+                if (!keyboard_interactively_grabbed())
+                    XUngrabKeyboard(ob_display, e->xkey.time);
+
                 action_run_key(p->actions, client, e->xkey.state,
                                e->xkey.x_root, e->xkey.y_root,
                                e->xkey.time);