1
0
Fork 0

wayland: Return boolean value handling pointer events

We've so far returned FALSE (i.e. PROPAGATE) here, somehow
oblivious of the fact that the core event handler would stop
all non-gesture events directed to windows.

Incorporate this knowledge there, in order to be able to
streamline this piece of event handling in core/ code.

Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/3420>
This commit is contained in:
Carlos Garnacho 2024-02-12 23:43:29 +01:00 committed by Robert Mader
parent c496b17b3d
commit 663f9d4495

View file

@ -844,34 +844,33 @@ meta_wayland_pointer_handle_event (MetaWaylandPointer *pointer,
{
case CLUTTER_MOTION:
handle_motion_event (pointer, event);
break;
return pointer->focus_surface ?
CLUTTER_EVENT_STOP : CLUTTER_EVENT_PROPAGATE;
case CLUTTER_BUTTON_PRESS:
case CLUTTER_BUTTON_RELEASE:
handle_button_event (pointer, event);
break;
return pointer->focus_surface ?
CLUTTER_EVENT_STOP : CLUTTER_EVENT_PROPAGATE;
case CLUTTER_SCROLL:
handle_scroll_event (pointer, event);
break;
return pointer->focus_surface ?
CLUTTER_EVENT_STOP : CLUTTER_EVENT_PROPAGATE;
case CLUTTER_TOUCHPAD_SWIPE:
meta_wayland_pointer_gesture_swipe_handle_event (pointer, event);
break;
return CLUTTER_EVENT_PROPAGATE;
case CLUTTER_TOUCHPAD_PINCH:
meta_wayland_pointer_gesture_pinch_handle_event (pointer, event);
break;
return meta_wayland_pointer_gesture_pinch_handle_event (pointer, event);
case CLUTTER_TOUCHPAD_HOLD:
meta_wayland_pointer_gesture_hold_handle_event (pointer, event);
break;
return meta_wayland_pointer_gesture_hold_handle_event (pointer, event);
default:
break;
return CLUTTER_EVENT_PROPAGATE;
}
return FALSE;
}
static void