This might be a little bit better.

At least it consistently doesn't run the openbox action.

diff --git a/openbox/event.c b/openbox/event.c
index a8e5c3f21b4ed5fddd3b68815b6a2d5be8a98977..8540373973399c0545b7a5fa7240b59ee1260f13 100644 (file)
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -740,10 +740,10 @@ static void event_process(const XEvent *ec, gpointer data)
             {
                 event_handle_user_input(client, e);
             }
-            /* If we have grabbed the mouse actively, I think this is how
-               to check it? */
-            else if (button)
-                event_handle_user_input(NULL, e);
+            /* We ignored the release event so make sure we don't think
+               the button is still pressed */
+            else if (e->type == ButtonRelease)
+                button = 0;
         }
     }
     else if (e->type == KeyPress || e->type == KeyRelease ||