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 83ac345d1e634c9bd3c0c5e910fe8b5365fb29d4..855ad0ff12cd8dcc0e35c66dc00935fa0a13ae93 100644 (file)
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -723,10 +723,10 @@ static void event_process(const XEvent *ec, gpointer data)
 
             if (e->type == ButtonPress)
                 pressed = e->xbutton.button;
-            /* 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 ||