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 eb2646b5d17694d187bee52ce1adafb9edba2e30..3b6d1743996282734dbfcae059fb9c4cd3eeb95d 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;
                 f->child == e->frame)
                 menu_frame_select(e->frame, e, FALSE);
         break;

-    case KeyPress:
+    case KeyRelease:

         ret = event_handle_menu_keyboard(ev);
         break;
     }
         ret = event_handle_menu_keyboard(ev);
         break;
     }