[PATCH weston] input: don't run the key bindings on focus in

Pekka Paalanen ppaalanen at gmail.com
Mon Nov 24 03:46:54 PST 2014


On Fri, 21 Nov 2014 13:45:04 +0000
Daniel Stone <daniel at fooishbar.org> wrote:

> Yep, this is correct.
> 
> (I'll get back to the other thread shortly.)
> 
> -d
> 
> On Thursday, November 20, 2014, Giulio Camuffo <giuliocamuffo at gmail.com>
> wrote:
> 
> > When getting the focus we get the list of pressed keys, but we are
> > not supposed to run the key binding on them.
> > ---
> >  src/input.c | 6 ------
> >  1 file changed, 6 deletions(-)
> >
> > diff --git a/src/input.c b/src/input.c
> > index 80aa34e..15ff6ed 100644
> > --- a/src/input.c
> > +++ b/src/input.c
> > @@ -1404,12 +1404,6 @@ notify_keyboard_focus_in(struct weston_seat *seat,
> > struct wl_array *keys,
> >
> > WL_KEYBOARD_KEY_STATE_PRESSED);
> >         }
> >
> > -       /* Run key bindings after we've updated the state. */
> > -       wl_array_for_each(k, &keyboard->keys) {
> > -               weston_compositor_run_key_binding(compositor, seat, 0, *k,
> > -
> >  WL_KEYBOARD_KEY_STATE_PRESSED);
> > -       }
> > -
> >         surface = seat->saved_kbd_focus;
> >
> >         if (surface) {
> > --

Pushed!

Thanks,
pq


More information about the wayland-devel mailing list