On Fri, Jun 26, 2015 at 12:37:55PM +0800, Jonas Ådahl wrote:
> In order to test warping when pointer confinement region changes, add
> key binding to the maximized state without using the mouse.
> 
> Signed-off-by: Jonas Ådahl <[email protected]>
> ---
> 
> This patch is an addition to the pointer lock series. Its purpose is to
> demonstrate the changing of the lock/confine regions without breaking
> the active or pending lock.
> 
> 
>  clients/clickdot.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)

I presume this patch depends on some other patches to land first, since
pointer_confined isn't yet in the clickdot structure.  But apart from
that detail, this LGTM for landing post-release:

Reviewed-by: Bryce Harrington <[email protected]>
 
> diff --git a/clients/clickdot.c b/clients/clickdot.c
> index bf5169e..b77c690 100644
> --- a/clients/clickdot.c
> +++ b/clients/clickdot.c
> @@ -309,6 +309,9 @@ key_handler(struct window *window, struct input *input, 
> uint32_t time,
>               clickdot->buffer = NULL;
>               window_schedule_redraw(clickdot->window);
>               break;
> +     case XKB_KEY_m:
> +             window_set_maximized(clickdot->window,
> +                                  !window_is_maximized(window));
>       }
>  }
>  
> @@ -388,6 +391,14 @@ resize_handler(struct widget *widget,
>  
>       clickdot->reset = 1;
>       clickdot->complex_confine_region_dirty = true;
> +
> +     if (clickdot->pointer_confined) {
> +             calculate_complex_confine_region(clickdot);
> +             window_update_confine_rectangles(
> +                             clickdot->window,
> +                             clickdot->complex_confine_region,
> +                             NUM_COMPLEX_REGION_RECTS);
> +     }
>  }
>  
>  static void
> -- 
> 2.1.4
> 
> _______________________________________________
> wayland-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to