Hi Quentin, On 22 March 2017 at 16:06, Quentin Glidic <sardemff7+wayl...@sardemff7.net> wrote: > @@ -2070,7 +2070,8 @@ notify_keyboard_focus_in(struct weston_seat *seat, > struct wl_array *keys, > > if (surface) { > wl_list_remove(&seat->saved_kbd_focus_listener.link); > - weston_keyboard_set_focus(keyboard, surface); > + if (!keyboard->focus) > + weston_keyboard_set_focus(keyboard, surface); > seat->saved_kbd_focus = NULL; > }
Would it not be better to just remove seat->saved_kbd_focus on weston_keyboard_set_focus()? That seems more correct to me regardless. Cheers, Daniel _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel