From c12991936efce9f01c74ec683f33a85e8f1d87c8 Mon Sep 17 00:00:00 2001 From: Carlos Garnacho Date: Wed, 28 Feb 2024 14:46:53 +0100 Subject: [PATCH] wayland: Let the tablet tool handle the handle_event() return value This was being ignored, and the events propagated always. This may lead to confusion with Clutter about who handled what, since shockingly the events may be delivered to clients. Closes: https://gitlab.gnome.org/GNOME/mutter/-/issues/3317 Part-of: --- src/wayland/meta-wayland-tablet-seat.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/wayland/meta-wayland-tablet-seat.c b/src/wayland/meta-wayland-tablet-seat.c index be4ee976f..308f42beb 100644 --- a/src/wayland/meta-wayland-tablet-seat.c +++ b/src/wayland/meta-wayland-tablet-seat.c @@ -435,8 +435,7 @@ meta_wayland_tablet_seat_handle_event (MetaWaylandTabletSeat *tablet_seat, if (!tool) return CLUTTER_EVENT_PROPAGATE; - meta_wayland_tablet_tool_handle_event (tool, event); - return CLUTTER_EVENT_PROPAGATE; + return meta_wayland_tablet_tool_handle_event (tool, event); case CLUTTER_PAD_BUTTON_PRESS: case CLUTTER_PAD_BUTTON_RELEASE: case CLUTTER_PAD_RING: