[PATCH weston 0/6] Better multi-seat zoom handling

Bryce Harrington bryce at osg.samsung.com
Tue Jan 27 14:33:16 PST 2015


Giulio or Jonas, would one of you mind doing a design review of this
patchset?

Thanks,
Bryce

On Fri, Jan 09, 2015 at 12:45:13PM -0600, Derek Foreman wrote:
> This started out as a small patch to stop weston from crashing when
> attempting to zoom with the key bind on a pointerless seat...
> 
> Zoom shouldn't crash when no pointer is present, and multiple seats
> shouldn't interfere with eachother's zoom anymore.
> 
> The new target picking semantics hopefully don't annoy anyone, but
> I think they only change things in cases that crashed or were
> obviously weird before.
> 
> The normal (single seat with a mouse) use case should act as before.
> 
> Derek Foreman (6):
>   zoom: Change zoom->active to a bool
>   zoom: Call weston_output_activate_zoom() appropriately
>   zoom: Rename zoom_area_center_from_pointer to
>     zoom_area_center_from_point
>   zoom: Store the seat that initiated a zoom
>   zoom: Only use a motion listener if a pointer is available
>   zoom: Change how we select a zoom location
> 
>  desktop-shell/shell.c | 25 +++++++++++-----
>  src/compositor.c      | 70 +++++++++++++++++++++++++++++++++++++++++++
>  src/compositor.h      | 16 ++++++++--
>  src/input.c           |  4 +++
>  src/zoom.c            | 82 +++++++++++++++++++++++++++++++++------------------
>  5 files changed, 160 insertions(+), 37 deletions(-)
> 
> -- 
> 2.1.4
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list