This might be a little bit better.
authorMikael Magnusson <mikachu@gmail.com>
Mon, 7 Jul 2008 20:34:55 +0000 (22:34 +0200)
committerMikael Magnusson <mikachu@gmail.com>
Thu, 6 Nov 2014 03:49:20 +0000 (04:49 +0100)
At least it consistently doesn't run the openbox action.

openbox/event.c

index 83ac345..855ad0f 100644 (file)
@@ -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 ||