diff options
| author | Balló György <[email protected]> | 2015-06-29 11:50:20 +0200 |
|---|---|---|
| committer | Balló György <[email protected]> | 2015-06-29 11:50:20 +0200 |
| commit | 2b8df1cf67516a9df1fb8931e88a06b452e890b5 (patch) | |
| tree | c08414969d51889e8b2afb45c17a6c052f2881de | |
| parent | e64c0d6d99fe6aef817af0b18e53d9813cb2c11f (diff) | |
| download | mate-panel-2b8df1cf67516a9df1fb8931e88a06b452e890b5.tar.bz2 mate-panel-2b8df1cf67516a9df1fb8931e88a06b452e890b5.tar.xz | |
Fix Force-Quit panel applet
This patch implements handling of XInput2 extension events in the applet.
Without this patch, the applet fails to recognize such events (either mouse
click for killing application, or escape key for exiting), hence freezing the
desktop.
Also fix the event mask in call of gdk_device_grab() for keyboard.
Author: Sébastien Villemot <[email protected]>
http://bugs.debian.org/698740
| -rw-r--r-- | mate-panel/panel-force-quit.c | 34 |
1 files changed, 27 insertions, 7 deletions
diff --git a/mate-panel/panel-force-quit.c b/mate-panel/panel-force-quit.c index 8231b8f1..f0374e2f 100644 --- a/mate-panel/panel-force-quit.c +++ b/mate-panel/panel-force-quit.c @@ -32,6 +32,8 @@ #include <X11/Xlib.h> #include <X11/keysym.h> +#include <X11/extensions/XInput2.h> + #include "panel-icon-names.h" #include "panel-stock-icons.h" @@ -261,22 +263,23 @@ kill_window_question (gpointer window) static void handle_button_press_event (GtkWidget *popup, - XKeyEvent *event) + Display *display, + Window subwindow) { Window window; remove_popup (popup); - if (event->subwindow == None) + if (subwindow == None) return; if (wm_state_atom == None) - wm_state_atom = XInternAtom (event->display, "WM_STATE", FALSE); + wm_state_atom = XInternAtom (display, "WM_STATE", FALSE); - window = find_managed_window (event->display, event->subwindow); + window = find_managed_window (display, subwindow); if (window != None) { - if (!gdk_x11_window_lookup_for_display (gdk_x11_lookup_xdisplay (event->display), window)) + if (!gdk_x11_window_lookup_for_display (gdk_x11_lookup_xdisplay (display), window)) kill_window_question ((gpointer) window); } } @@ -287,10 +290,12 @@ popup_filter (GdkXEvent *gdk_xevent, GtkWidget *popup) { XEvent *xevent = (XEvent *) gdk_xevent; + XIEvent *xiev; + XIDeviceEvent *xidev; switch (xevent->type) { case ButtonPress: - handle_button_press_event (popup, &xevent->xkey); + handle_button_press_event (popup, xevent->xbutton.display, xevent->xbutton.subwindow); return GDK_FILTER_REMOVE; case KeyPress: if (xevent->xkey.keycode == XKeysymToKeycode (xevent->xany.display, XK_Escape)) { @@ -298,6 +303,21 @@ popup_filter (GdkXEvent *gdk_xevent, return GDK_FILTER_REMOVE; } break; + case GenericEvent: + xiev = (XIEvent *) xevent->xcookie.data; + xidev = (XIDeviceEvent *) xiev; + switch (xiev->evtype) { + case XI_KeyPress: + if (xidev->detail == XKeysymToKeycode (xevent->xany.display, XK_Escape)) { + remove_popup (popup); + return GDK_FILTER_REMOVE; + } + break; + case XI_ButtonPress: + handle_button_press_event (popup, xidev->display, xidev->child); + return GDK_FILTER_REMOVE; + } + break; default: break; } @@ -342,7 +362,7 @@ panel_force_quit (GdkScreen *screen, status = gdk_device_grab (keyboard, root, GDK_OWNERSHIP_NONE, FALSE, - GDK_KEY_PRESS | GDK_KEY_RELEASE, + GDK_KEY_PRESS_MASK | GDK_KEY_RELEASE_MASK, NULL, time); if (status != GDK_GRAB_SUCCESS) { |