On Wed, Sep 26, 2012 at 03:05:45PM +0300, Pekka Vuorela wrote:
> As common UI convention, allows action to be avoided by dragging
> outside the button after mouse press.

That works better, thanks.

Kristian

> ---
>  clients/window.c |   32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
> 
> diff --git a/clients/window.c b/clients/window.c
> index c5bccdc..f3b61de 100644
> --- a/clients/window.c
> +++ b/clients/window.c
> @@ -1498,6 +1498,7 @@ frame_button_button_handler(struct widget *widget,
>  {
>       struct frame_button *frame_button = data;
>       struct window *window = widget->window;
> +     int was_pressed = (frame_button->state == FRAME_BUTTON_ACTIVE);
>  
>       if (button != BTN_LEFT)
>               return;
> @@ -1516,6 +1517,9 @@ frame_button_button_handler(struct widget *widget,
>               break;
>       }
>  
> +     if (!was_pressed)
> +             return;
> +
>       switch (frame_button->type) {
>       case FRAME_BUTTON_CLOSE:
>               if (window->close_handler)
> @@ -1536,6 +1540,33 @@ frame_button_button_handler(struct widget *widget,
>       }
>  }
>  
> +static int
> +frame_button_motion_handler(struct widget *widget,
> +                            struct input *input, uint32_t time,
> +                            float x, float y, void *data)
> +{
> +     struct frame_button *frame_button = data;
> +     enum frame_button_pointer previous_button_state = frame_button->state;
> +
> +     /* only track state for a pressed button */
> +     if (input->grab != widget)
> +             return CURSOR_LEFT_PTR;
> +
> +     if (x > widget->allocation.x &&
> +         x < (widget->allocation.x + widget->allocation.width) &&
> +         y > widget->allocation.y &&
> +         y < (widget->allocation.y + widget->allocation.height)) {
> +             frame_button->state = FRAME_BUTTON_ACTIVE;
> +     } else {
> +             frame_button->state = FRAME_BUTTON_DEFAULT;
> +     }
> +
> +     if (frame_button->state != previous_button_state)
> +             widget_schedule_redraw(frame_button->widget);
> +
> +     return CURSOR_LEFT_PTR;
> +}
> +
>  static void
>  frame_button_redraw_handler(struct widget *widget, void *data)
>  {
> @@ -1612,6 +1643,7 @@ frame_button_create(struct frame *frame, void *data, 
> enum frame_button_action ty
>       widget_set_enter_handler(frame_button->widget, 
> frame_button_enter_handler);
>       widget_set_leave_handler(frame_button->widget, 
> frame_button_leave_handler);
>       widget_set_button_handler(frame_button->widget, 
> frame_button_button_handler);
> +     widget_set_motion_handler(frame_button->widget, 
> frame_button_motion_handler);
>       return frame_button->widget;
>  }
>  
> -- 
> 1.7.9.5
> 
> _______________________________________________
> 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