diff options
| author | Jasper St. Pierre <jstpierre@mecheye.net> | 2014-08-15 13:12:22 -0400 |
|---|---|---|
| committer | Jasper St. Pierre <jstpierre@mecheye.net> | 2014-08-15 16:08:49 -0400 |
| commit | 0e758a9e655fa2658b9e2eef834f68e42d217419 (patch) | |
| tree | 933ee1b40c779d50b66d9216f0d2cf017ba351a6 /src/backends/meta-cursor-tracker.c | |
| parent | 64a915a68ddd492709474db3e89c80c847e94ebe (diff) | |
| download | mutter-0e758a9e655fa2658b9e2eef834f68e42d217419.tar.gz | |
display: Establish a separate state variable for routing events
We've long used a switch statement on the grab operation to determine
where events should go. The issue with MetaGrabOp is that it's a mixture
of a few different things, including event routing, state management,
and the behavior to choose during operations.
This leads to poorly defined event routing and hard-to-follow logic,
since it's sometimes unclear what should point where, and our utility
methods for determining grab operations apart can be poorly named.
To fix this, establish the concept of a "event route", which describes
where events should be routed to.
Diffstat (limited to 'src/backends/meta-cursor-tracker.c')
| -rw-r--r-- | src/backends/meta-cursor-tracker.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backends/meta-cursor-tracker.c b/src/backends/meta-cursor-tracker.c index 13991457e..a649cb47d 100644 --- a/src/backends/meta-cursor-tracker.c +++ b/src/backends/meta-cursor-tracker.c @@ -61,7 +61,7 @@ get_displayed_cursor (MetaCursorTracker *tracker) if (!tracker->is_showing) return NULL; - if (meta_grab_op_windows_are_interactable (display->grab_op)) + if (meta_display_windows_are_interactable (display)) { if (tracker->has_window_cursor) return tracker->window_cursor; |