use keyrelease to fire menu actions not key press

when keypress is used, the keyrelease gets sent to whatever window is focused.  if it's a stupid program - like most any program - it will update its user_time, preventing the new window from being focused

diff --git a/openbox/event.c b/openbox/event.c
index eb2646b..3b6d174 100644 (file)
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -1778,7 +1778,7 @@ static gboolean event_handle_menu(XEvent *ev)
                 f->child == e->frame)
                 menu_frame_select(e->frame, e, FALSE);
         break;
-    case KeyPress:
+    case KeyRelease:
         ret = event_handle_menu_keyboard(ev);
         break;
     }