mouse: Pointer locater improvements

Make it work on Gtk3 and fix clicking through on Gtk2.
author: Alexei Sorokin <[email protected]> 2017-03-15 17:20:21 +0300
committer: Alexei Sorokin <[email protected]> 2017-03-15 17:20:21 +0300
commit: 853cc71a92cb5e931f2a85b58cf6f3e2ba6bc2c2 (patch)
tree: 5df7388118c58e2e5c47ef118f4d634f089c4071
parent: e686dbbf8bfd5e91c8981e8aab33c11b44cb5208 (diff)
download: mate-settings-daemon-853cc71a92cb5e931f2a85b58cf6f3e2ba6bc2c2.tar.bz2
mate-settings-daemon-853cc71a92cb5e931f2a85b58cf6f3e2ba6bc2c2.tar.xz
1 files changed, 270 insertions, 153 deletions
diff --git a/plugins/mouse/msd-locate-pointer.c b/plugins/mouse/msd-locate-pointer.c
index a480a6a..468d840 100644
--- a/plugins/mouse/msd-locate-pointer.c
+++ b/plugins/mouse/msd-locate-pointer.c
@@ -42,7 +42,7 @@ typedef struct MsdLocatePointerData MsdLocatePointerData;
 struct MsdLocatePointerData
 {
 MsdTimeline *timeline;
- GtkWidget *widget;
+ GtkWindow *widget;
 GdkWindow *window;
 
 gdouble progress;
@@ -50,41 +50,26 @@ struct MsdLocatePointerData
 
 static MsdLocatePointerData *data = NULL;
 
-#if GTK_CHECK_VERSION(3, 0, 0)
-static void
-msd_get_background_color (GtkStyleContext *context,
- GtkStateFlags state,
- GdkRGBA *color)
-{
- GdkRGBA *c;
-
- g_return_if_fail (color != NULL);
- g_return_if_fail (GTK_IS_STYLE_CONTEXT (context));
-
- gtk_style_context_get (context,
- state,
- "background-color", &c,
- NULL);
- *color = *c;
- gdk_rgba_free (c);
-}
-#endif
-
 static void
 locate_pointer_paint (MsdLocatePointerData *data,
 cairo_t *cr,
- gboolean composite)
+ gboolean composited)
 {
 #if GTK_CHECK_VERSION (3, 0, 0)
 GdkRGBA color;
 gdouble progress, circle_progress;
 gint width, height, i;
- GtkStyleContext *context;
+ GtkStyleContext *style;
+
+ color.red = color.green = color.blue = 0.7;
+ color.alpha = 0.;
 #else
 GdkColor color;
 gdouble progress, circle_progress;
 gint width, height, i;
 GtkStyle *style;
+
+ color.red = color.green = color.blue = 0xAAAA;
 #endif
 
 progress = data->progress;
@@ -93,15 +78,29 @@ locate_pointer_paint (MsdLocatePointerData *data,
 height = gdk_window_get_height (data->window);
 
 #if GTK_CHECK_VERSION (3, 0, 0)
- context = gtk_widget_get_style_context (data->widget);
- msd_get_background_color (context, GTK_STATE_FLAG_SELECTED, &color);
+ style = gtk_widget_get_style_context (GTK_WIDGET (data->widget));
+ gtk_style_context_save (style);
+ gtk_style_context_set_state (style, GTK_STATE_FLAG_SELECTED);
+ gtk_style_context_add_class (style, GTK_STYLE_CLASS_VIEW);
+ gtk_style_context_get_background_color (style,
+ gtk_style_context_get_state (style),
+ &color);
+ if (color.alpha == 0.)
+ {
+ gtk_style_context_remove_class (style, GTK_STYLE_CLASS_VIEW);
+ gtk_style_context_get_background_color (style,
+ gtk_style_context_get_state (style),
+ &color);
+ }
+ gtk_style_context_restore (style);
 #else
- style = gtk_widget_get_style (data->widget);
+ style = gtk_widget_get_style (GTK_WIDGET (data->widget));
 color = style->bg[GTK_STATE_SELECTED];
 #endif
 
- cairo_set_source_rgba (cr, 1., 1., 1., 0.);
+ cairo_save (cr);
 cairo_set_operator (cr, CAIRO_OPERATOR_SOURCE);
+ cairo_set_source_rgba (cr, 1., 1., 1., 0.);
 cairo_paint (cr);
 
 for (i = 0; i <= N_CIRCLES; i++)
@@ -115,7 +114,7 @@ locate_pointer_paint (MsdLocatePointerData *data,
 if (circle_progress >= 1.)
 continue;
 
- if (composite)
+ if (composited)
 {
 cairo_set_source_rgba (cr,
 #if GTK_CHECK_VERSION (3, 0, 0)
@@ -158,58 +157,31 @@ locate_pointer_paint (MsdLocatePointerData *data,
 cairo_stroke (cr);
 }
 }
+ cairo_restore (cr);
 }
 
-#if GTK_CHECK_VERSION (3, 0, 0)
-static gboolean
-locate_pointer_draw (GtkWidget *widget,
- cairo_t *cr,
- gpointer user_data)
-{
- MsdLocatePointerData *data = (MsdLocatePointerData *) user_data;
-
- if (gtk_cairo_should_draw_window (cr, data->window))
- locate_pointer_paint (data, cr, gtk_widget_is_composited (data->widget));
-
- return TRUE;
-}
-#else
-static gboolean
-locate_pointer_expose (GtkWidget *widget,
- GdkEventExpose *event,
- gpointer user_data)
-{
- MsdLocatePointerData *data = (MsdLocatePointerData *) user_data;
- cairo_t *cr;
-
- if (event->window != data->window)
- return FALSE;
-
- cr = gdk_cairo_create (data->window);
- locate_pointer_paint (data, cr, gtk_widget_is_composited (data->widget));
- cairo_destroy (cr);
-
- return TRUE;
-}
-#endif
-
 static void
 update_shape (MsdLocatePointerData *data)
 {
 cairo_t *cr;
 #if GTK_CHECK_VERSION (3, 0, 0)
- cairo_region_t *region;
 cairo_surface_t *mask;
+ cairo_region_t *region;
 
- mask = cairo_image_surface_create (CAIRO_FORMAT_A1, WINDOW_SIZE, WINDOW_SIZE);
+ mask = gdk_window_create_similar_image_surface (data->window,
+ CAIRO_FORMAT_A1,
+ WINDOW_SIZE,
+ WINDOW_SIZE,
+ 0);
 cr = cairo_create (mask);
- region = gdk_cairo_region_create_from_surface (mask);
 
 locate_pointer_paint (data, cr, FALSE);
 
+ region = gdk_cairo_region_create_from_surface (mask);
+
 gdk_window_shape_combine_region (data->window, region, 0, 0);
- cairo_region_destroy (region);
 
+ cairo_region_destroy (region);
 cairo_destroy (cr);
 cairo_surface_destroy (mask);
 #else
@@ -221,9 +193,9 @@ update_shape (MsdLocatePointerData *data)
 locate_pointer_paint (data, cr, FALSE);
 
 gdk_window_shape_combine_mask (data->window, mask, 0, 0);
- g_object_unref (mask);
 
 cairo_destroy (cr);
+ g_object_unref (mask);
 #endif
 }
 
@@ -233,40 +205,48 @@ timeline_frame_cb (MsdTimeline *timeline,
 gpointer user_data)
 {
 MsdLocatePointerData *data = (MsdLocatePointerData *) user_data;
- GdkScreen *screen;
-#if GTK_CHECK_VERSION (3, 0, 0)
+ GdkScreen *screen = gdk_window_get_screen (data->window);
+#if GTK_CHECK_VERSION (3, 20, 0)
+ GdkDisplay *display = gdk_window_get_display (data->window);
+ GdkSeat *seat;
+ GdkDevice *pointer;
+#elif GTK_CHECK_VERSION (3, 0, 0)
+ GdkDisplay *display = gdk_window_get_display (data->window);
 GdkDeviceManager *device_manager;
 GdkDevice *pointer;
 #endif
 gint cursor_x, cursor_y;
 
- if (gtk_widget_is_composited (data->widget))
+ if (gdk_screen_is_composited (screen))
 {
- gdk_window_invalidate_rect (data->window, NULL, FALSE);
+ gtk_widget_queue_draw (GTK_WIDGET (data->widget));
 data->progress = progress;
 }
 else if (progress >= data->progress + CIRCLES_PROGRESS_INTERVAL)
 {
 /* only invalidate window each circle interval */
 update_shape (data);
- gdk_window_invalidate_rect (data->window, NULL, FALSE);
+ gtk_widget_queue_draw (GTK_WIDGET (data->widget));
 data->progress += CIRCLES_PROGRESS_INTERVAL;
 }
 
- screen = gdk_window_get_screen (data->window);
 #if GTK_CHECK_VERSION (3, 0, 0)
- device_manager = gdk_display_get_device_manager (gdk_window_get_display (gdk_screen_get_root_window (screen)));
+#if GTK_CHECK_VERSION (3, 20, 0)
+ seat = gdk_display_get_default_seat (display);
+ pointer = gdk_seat_get_pointer (seat);
+#else
+ device_manager = gdk_display_get_device_manager (display);
 pointer = gdk_device_manager_get_client_pointer (device_manager);
- gdk_window_get_device_position (gdk_screen_get_root_window (screen),
- pointer,
- &cursor_x,
- &cursor_y,
- NULL);
+#endif
+ gdk_device_get_position (pointer,
+ NULL,
+ &cursor_x,
+ &cursor_y);
 #else
 gdk_window_get_pointer (gdk_screen_get_root_window (screen),
 &cursor_x, &cursor_y, NULL);
 #endif
- gdk_window_move (data->window,
+ gtk_window_move (data->widget,
 cursor_x - WINDOW_SIZE / 2,
 cursor_y - WINDOW_SIZE / 2);
 }
@@ -275,26 +255,41 @@ static void
 set_transparent_shape (GdkWindow *window)
 {
 #if GTK_CHECK_VERSION (3, 0, 0)
+ cairo_t *cr;
+ cairo_surface_t *mask;
 cairo_region_t *region;
 
- region = cairo_region_create ();
-
- gdk_window_input_shape_combine_region (data->window, region, 0, 0);
- cairo_region_destroy (region);
+ mask = gdk_window_create_similar_image_surface (window,
+ CAIRO_FORMAT_A1,
+ WINDOW_SIZE,
+ WINDOW_SIZE,
+ 0);
+ cr = cairo_create (mask);
 #else
- GdkBitmap *mask;
 cairo_t *cr;
+ GdkBitmap *mask;
 
 mask = gdk_pixmap_new (data->window, WINDOW_SIZE, WINDOW_SIZE, 1);
 cr = gdk_cairo_create (mask);
+#endif
 
 cairo_set_source_rgba (cr, 1., 1., 1., 0.);
 cairo_set_operator (cr, CAIRO_OPERATOR_SOURCE);
 cairo_paint (cr);
 
+#if GTK_CHECK_VERSION (3, 0, 0)
+ region = gdk_cairo_region_create_from_surface (mask);
+
+ gdk_window_shape_combine_region (window, region, 0, 0);
+
+ cairo_region_destroy (region);
+ cairo_destroy (cr);
+ cairo_surface_destroy (mask);
+#else
 gdk_window_shape_combine_mask (data->window, mask, 0, 0);
- g_object_unref (mask);
+
 cairo_destroy (cr);
+ g_object_unref (mask);
 #endif
 }
 
@@ -302,20 +297,24 @@ static void
 unset_transparent_shape (GdkWindow *window)
 {
 #if GTK_CHECK_VERSION (3, 0, 0)
- gdk_window_shape_combine_region (data->window, NULL, 0, 0);
+ gdk_window_shape_combine_region (window, NULL, 0, 0);
 #else
- gdk_window_shape_combine_mask (data->window, NULL, 0, 0);
+ gdk_window_shape_combine_mask (window, NULL, 0, 0);
 #endif
 }
 
 static void
-composited_changed (GtkWidget *widget,
+composited_changed (GdkScreen *screen,
 MsdLocatePointerData *data)
 {
- if (!gtk_widget_is_composited (widget))
- set_transparent_shape (data->window);
+ if (gdk_screen_is_composited (screen))
+ {
+ unset_transparent_shape (data->window);
+ }
 else
- unset_transparent_shape (data->window);
+ {
+ set_transparent_shape (data->window);
+ }
 }
 
 static void
@@ -323,18 +322,39 @@ timeline_finished_cb (MsdTimeline *timeline,
 gpointer user_data)
 {
 MsdLocatePointerData *data = (MsdLocatePointerData *) user_data;
+ GdkScreen *screen = gdk_window_get_screen (data->window);
 
 /* set transparent shape and hide window */
- if (!gtk_widget_is_composited (data->widget))
- set_transparent_shape (data->window);
+ if (!gdk_screen_is_composited (screen))
+ {
+ set_transparent_shape (data->window);
+ }
 
- gdk_window_hide (data->window);
+ gtk_widget_hide (GTK_WIDGET (data->widget));
 }
 
 static void
-create_window (MsdLocatePointerData *data,
- GdkScreen *screen)
+locate_pointer_unrealize_cb (GtkWidget *widget,
+ MsdLocatePointerData *data)
 {
+ if (data->window != NULL)
+ {
+#if GTK_CHECK_VERSION (3, 0, 0)
+ gtk_widget_unregister_window (GTK_WIDGET (data->widget),
+ data->window);
+#else
+ gdk_window_set_user_data (data->window, NULL);
+#endif
+ gdk_window_destroy (data->window);
+ }
+ data->window = NULL;
+}
+
+static void
+locate_pointer_realize_cb (GtkWidget *widget,
+ MsdLocatePointerData *data)
+{
+ GdkScreen *screen;
 #if !GTK_CHECK_VERSION (3, 0, 0)
 GdkColormap *colormap;
 #endif
@@ -342,32 +362,39 @@ create_window (MsdLocatePointerData *data,
 GdkWindowAttr attributes;
 gint attributes_mask;
 
+ screen = gtk_window_get_screen (data->widget);
 #if !GTK_CHECK_VERSION (3, 0, 0)
 colormap = gdk_screen_get_rgba_colormap (screen);
 #endif
 visual = gdk_screen_get_rgba_visual (screen);
 
-#if GTK_CHECK_VERSION (3, 0, 0)
+#if !GTK_CHECK_VERSION (3, 0, 0)
+ if (colormap == NULL)
+ colormap = gdk_screen_get_system_colormap (screen);
+#endif
 if (visual == NULL)
 visual = gdk_screen_get_system_visual (screen);
-#endif
 
- attributes_mask = GDK_WA_X | GDK_WA_Y;
+ locate_pointer_unrealize_cb (GTK_WIDGET (data->widget), data);
 
-#if GTK_CHECK_VERSION (3, 0, 0)
- if (visual != NULL)
- attributes_mask = attributes_mask | GDK_WA_VISUAL;
-#else
- if (colormap)
- attributes_mask = attributes_mask | GDK_WA_VISUAL | GDK_WA_COLORMAP;
+ attributes_mask = GDK_WA_X | GDK_WA_Y;
+#if !GTK_CHECK_VERSION (3, 0, 0)
+ if (colormap != NULL)
+ {
+ attributes_mask |= GDK_WA_COLORMAP;
+ }
 #endif
+ if (visual != NULL)
+ {
+ attributes_mask |= GDK_WA_VISUAL;
+ }
 
 attributes.window_type = GDK_WINDOW_TEMP;
 attributes.wclass = GDK_INPUT_OUTPUT;
- attributes.visual = visual;
 #if !GTK_CHECK_VERSION (3, 0, 0)
 attributes.colormap = colormap;
 #endif
+ attributes.visual = visual;
 attributes.event_mask = GDK_VISIBILITY_NOTIFY_MASK | GDK_EXPOSURE_MASK;
 attributes.width = 1;
 attributes.height = 1;
@@ -376,9 +403,59 @@ create_window (MsdLocatePointerData *data,
 &attributes,
 attributes_mask);
 
- gdk_window_set_user_data (data->window, data->widget);
+ gtk_widget_set_window (GTK_WIDGET (data->widget),
+ data->window);
+#if GTK_CHECK_VERSION (3, 0, 0)
+ gtk_widget_register_window (GTK_WIDGET (data->widget),
+ data->window);
+#else
+ gdk_window_set_user_data (data->window, GTK_WIDGET (data->widget));
+#endif
 }
 
+#if GTK_CHECK_VERSION (3, 0, 0)
+static gboolean
+locate_pointer_draw_cb (GtkWidget *widget,
+ cairo_t *cr,
+ gpointer user_data)
+{
+ MsdLocatePointerData *data = (MsdLocatePointerData *) user_data;
+ GdkScreen *screen = gdk_window_get_screen (data->window);
+
+ if (gtk_cairo_should_draw_window (cr, data->window))
+ {
+ locate_pointer_paint (data, cr, gdk_screen_is_composited (screen));
+ }
+
+ return TRUE;
+}
+#else
+static gboolean
+locate_pointer_expose_event_cb (GtkWidget *widget,
+ GdkEventExpose *event,
+ gpointer user_data)
+{
+ MsdLocatePointerData *data = (MsdLocatePointerData *) user_data;
+ GdkScreen *screen = gdk_window_get_screen (data->window);
+ cairo_t *cr;
+
+ if (event->window != data->window)
+ {
+ return FALSE;
+ }
+
+ cr = gdk_cairo_create (data->window);
+
+ gdk_cairo_rectangle (cr, &event->area);
+ cairo_clip (cr);
+
+ locate_pointer_paint (data, cr, gdk_screen_is_composited (screen));
+
+ cairo_destroy (cr);
+ return TRUE;
+}
+#endif
+
 static MsdLocatePointerData *
 msd_locate_pointer_data_new (GdkScreen *screen)
 {
@@ -389,17 +466,27 @@ msd_locate_pointer_data_new (GdkScreen *screen)
 /* this widget will never be shown, it's
 * mainly used to get signals/events from
 */
- data->widget = gtk_window_new (GTK_WINDOW_POPUP);
- gtk_widget_realize (data->widget);
+ data->widget = GTK_WINDOW (gtk_window_new (GTK_WINDOW_POPUP));
 
+ g_signal_connect (GTK_WIDGET (data->widget), "unrealize",
+ G_CALLBACK (locate_pointer_unrealize_cb),
+ data);
+ g_signal_connect (GTK_WIDGET (data->widget), "realize",
+ G_CALLBACK (locate_pointer_realize_cb),
+ data);
 #if GTK_CHECK_VERSION (3, 0, 0)
 g_signal_connect (G_OBJECT (data->widget), "draw",
- G_CALLBACK (locate_pointer_draw),
+ G_CALLBACK (locate_pointer_draw_cb),
+ data);
 #else
- g_signal_connect (G_OBJECT (data->widget), "expose_event",
- G_CALLBACK (locate_pointer_expose),
-#endif
+ g_signal_connect (G_OBJECT (data->widget), "expose-event",
+ G_CALLBACK (locate_pointer_expose_event_cb),
 data);
+#endif
+
+ gtk_window_set_screen (data->widget, screen);
+ gtk_widget_set_app_paintable (GTK_WIDGET (data->widget), TRUE);
+ gtk_widget_realize (GTK_WIDGET (data->widget));
 
 data->timeline = msd_timeline_new (ANIMATION_LENGTH);
 g_signal_connect (data->timeline, "frame",
@@ -407,8 +494,6 @@ msd_locate_pointer_data_new (GdkScreen *screen)
 g_signal_connect (data->timeline, "finished",
 G_CALLBACK (timeline_finished_cb), data);
 
- create_window (data, screen);
-
 return data;
 }
 
@@ -416,48 +501,80 @@ static void
 move_locate_pointer_window (MsdLocatePointerData *data,
 GdkScreen *screen)
 {
-#if GTK_CHECK_VERSION (3, 0, 0)
+#if GTK_CHECK_VERSION (3, 20, 0)
+ GdkDisplay *display;
+ GdkSeat *seat;
+ GdkDevice *pointer;
+#elif GTK_CHECK_VERSION (3, 0, 0)
+ GdkDisplay *display;
 GdkDeviceManager *device_manager;
 GdkDevice *pointer;
 #endif
 gint cursor_x, cursor_y;
-#if !GTK_CHECK_VERSION (3, 0, 0)
- GdkBitmap *mask;
 cairo_t *cr;
+#if GTK_CHECK_VERSION (3, 0, 0)
+ cairo_surface_t *mask;
+ cairo_region_t *region;
+#else
+ GdkBitmap *mask;
 #endif
 
 #if GTK_CHECK_VERSION (3, 0, 0)
- device_manager = gdk_display_get_device_manager (gdk_window_get_display (gdk_screen_get_root_window (screen)));
+ display = gdk_screen_get_display (screen);
+#if GTK_CHECK_VERSION (3, 20, 0)
+ seat = gdk_display_get_default_seat (display);
+ pointer = gdk_seat_get_pointer (seat);
+#else
+ device_manager = gdk_display_get_device_manager (display);
 pointer = gdk_device_manager_get_client_pointer (device_manager);
- gdk_window_get_device_position (gdk_screen_get_root_window (screen),
- pointer,
- &cursor_x,
- &cursor_y,
- NULL);
+#endif
+ gdk_device_get_position (pointer,
+ NULL,
+ &cursor_x,
+ &cursor_y);
 #else
- gdk_window_get_pointer (gdk_screen_get_root_window (screen), &cursor_x, &cursor_y, NULL);
+ gdk_window_get_pointer (gdk_screen_get_root_window (screen),
+ &cursor_x,
+ &cursor_y,
+ NULL);
 #endif
 
- gdk_window_move_resize (data->window,
- cursor_x - WINDOW_SIZE / 2,
- cursor_y - WINDOW_SIZE / 2,
- WINDOW_SIZE, WINDOW_SIZE);
+ gtk_window_move (data->widget,
+ cursor_x - WINDOW_SIZE / 2,
+ cursor_y - WINDOW_SIZE / 2);
+ gtk_window_resize (data->widget,
+ WINDOW_SIZE, WINDOW_SIZE);
 
 #if GTK_CHECK_VERSION (3, 0, 0)
- gdk_window_input_shape_combine_region (data->window, NULL, 0, 0);
+ mask = gdk_window_create_similar_image_surface (data->window,
+ CAIRO_FORMAT_A1,
+ WINDOW_SIZE,
+ WINDOW_SIZE,
+ 0);
+ cr = cairo_create (mask);
+
+ cairo_set_source_rgba (cr, 0., 0., 0., 0.);
+ cairo_paint (cr);
+
+ region = gdk_cairo_region_create_from_surface (mask);
+
+ /* allow events to happen through the window */
+ gdk_window_input_shape_combine_region (data->window, region, 0, 0);
+
+ cairo_region_destroy (region);
+ cairo_destroy (cr);
+ cairo_surface_destroy (mask);
 #else
 mask = gdk_pixmap_new (data->window, WINDOW_SIZE, WINDOW_SIZE, 1);
-
 cr = gdk_cairo_create (mask);
- cairo_set_source_rgb (cr, 0., 0., 0.);
- cairo_rectangle (cr, 0., 0., WINDOW_SIZE, WINDOW_SIZE);
- cairo_fill (cr);
- cairo_destroy (cr);
 
+ cairo_set_source_rgba (cr, 0., 0., 0., 0.);
+ cairo_paint (cr);
 
 /* allow events to happen through the window */
 gdk_window_input_shape_combine_mask (data->window, mask, 0, 0);
 
+ cairo_destroy (cr);
 g_object_unref (mask);
 #endif
 }
@@ -465,8 +582,10 @@ move_locate_pointer_window (MsdLocatePointerData *data,
 void
 msd_locate_pointer (GdkScreen *screen)
 {
- if (!data)
- data = msd_locate_pointer_data_new (screen);
+ if (data == NULL)
+ {
+ data = msd_locate_pointer_data_new (screen);
+ }
 
 msd_timeline_pause (data->timeline);
 msd_timeline_rewind (data->timeline);
@@ -474,20 +593,19 @@ msd_locate_pointer (GdkScreen *screen)
 /* Create again the window if it is not for the current screen */
 if (gdk_screen_get_number (screen) != gdk_screen_get_number (gdk_window_get_screen (data->window)))
 {
- gdk_window_set_user_data (data->window, NULL);
- gdk_window_destroy (data->window);
-
- create_window (data, screen);
+ gtk_widget_unrealize (GTK_WIDGET (data->widget));
+ gtk_window_set_screen (data->widget, screen);
+ gtk_widget_realize (GTK_WIDGET (data->widget));
 }
 
 data->progress = 0.;
 
- g_signal_connect (data->widget, "composited-changed",
+ g_signal_connect (screen, "composited-changed",
 G_CALLBACK (composited_changed), data);
 
 move_locate_pointer_window (data, screen);
- composited_changed (data->widget, data);
- gdk_window_show (data->window);
+ composited_changed (screen, data);
+ gtk_widget_show (GTK_WIDGET (data->widget));
 
 msd_timeline_start (data->timeline);
 }
@@ -502,11 +620,14 @@ filter (GdkXEvent *xevent,
 GdkEvent *event,
 gpointer data)
 {
+ GdkScreen *screen;
+ GdkDisplay *display;
 XEvent *xev = (XEvent *) xevent;
 guint keyval;
 gint group;
 
- GdkScreen *screen = (GdkScreen *)data;
+ screen = (GdkScreen *)data;
+ display = gdk_screen_get_display (screen);
 
 if (xev->type == KeyPress || xev->type == KeyRelease)
 {
@@ -539,7 +660,7 @@ filter (GdkXEvent *xevent,
 XAllowEvents (xev->xkey.display,
 ReplayKeyboard,
 xev->xkey.time);
- XUngrabKeyboard (gdk_x11_get_default_xdisplay (),
+ XUngrabKeyboard (GDK_DISPLAY_XDISPLAY (display),
 xev->xkey.time);
 }
 }
@@ -556,7 +677,7 @@ set_locate_pointer (void)
 int n_keys;
 gboolean has_entries;
 static const guint keyvals[] = { GDK_KEY_Control_L, GDK_KEY_Control_R };
- unsigned j;
+ unsigned int j;
 
 display = gdk_display_get_default ();
 n_screens = gdk_display_get_n_screens (display);
@@ -572,17 +693,13 @@ set_locate_pointer (void)
 gint i, j;
 for (i = 0; i < n_keys; i++)
 {
- for (j=0; j< n_screens; j++)
+ for (j = 0; j < n_screens; j++)
 {
 GdkScreen *screen;
 Window xroot;
 
 screen = gdk_display_get_screen (display, j);
-#if GTK_CHECK_VERSION (3, 0, 0)
- xroot = gdk_x11_window_get_xid (gdk_screen_get_root_window (screen));
-#else
- xroot = gdk_x11_drawable_get_xid (gdk_screen_get_root_window (screen));
-#endif
+ xroot = GDK_WINDOW_XID (gdk_screen_get_root_window (screen));
 
 XGrabKey (GDK_DISPLAY_XDISPLAY (display),
 keys[i].keycode,
author	Alexei Sorokin <[email protected]>	2017-03-15 17:20:21 +0300
committer	Alexei Sorokin <[email protected]>	2017-03-15 17:20:21 +0300
commit	853cc71a92cb5e931f2a85b58cf6f3e2ba6bc2c2 (patch)
tree	5df7388118c58e2e5c47ef118f4d634f089c4071
parent	e686dbbf8bfd5e91c8981e8aab33c11b44cb5208 (diff)
download	mate-settings-daemon-853cc71a92cb5e931f2a85b58cf6f3e2ba6bc2c2.tar.bz2 mate-settings-daemon-853cc71a92cb5e931f2a85b58cf6f3e2ba6bc2c2.tar.xz