<div dir="auto"><div dir="auto"><div dir="auto">Hey Daniel, thanks for taking a look! It turns out that this patch worked in Weston 3, but in Weston 4, commit 85d55540cb64bf97a08b40f79dc66843f8295d3 broke it. I didn't investigate carefully enough to understand what that commit was for or why it broke this, but my patch plus a revert of that commit makes Weston 4 work without side effects we've been able to observe, so that combo is what we've been using for the last month.</div><div dir="auto"><br></div><div dir="auto">I hope that's useful information…</div><div dir="auto">Jamey</div><br><div class="gmail_quote"><div dir="ltr">On Sun, Jul 22, 2018, 3:09 AM Daniel Stone <<a href="mailto:daniel@fooishbar.org" rel="noreferrer noreferrer" target="_blank">daniel@fooishbar.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jamey,<br>
<br>
On Wed, 23 May 2018 at 14:16, Jamey Sharp <<a href="mailto:jamey@minilop.net" rel="noreferrer noreferrer noreferrer" target="_blank">jamey@minilop.net</a>> wrote:<br>
> Under Weston's drm-backend, after returning to Weston from another VT,<br>
> no window has focus.<br>
><br>
> There's already code in notify_keyboard_focus_out and<br>
> notify_keyboard_focus_in to save and restore focus, respectively; and<br>
> udev_input_enable eventually calls notify_keyboard_focus_in, by way of<br>
> evdev_notify_keyboard_focus. But udev_input_disable doesn't currently<br>
> call notify_keyboard_focus_out.<br>
><br>
> This patch makes udev_input_disable symmetric with udev_input_enable by<br>
> calling notify_keyboard_focus_out on every seat, to save focus before<br>
> suspending libinput. In my testing this successfully resolved my issue.<br>
> However, I don't have a multi-seat setup to verify that it works there<br>
> too.<br>
<br>
This seems correct to me. Peter, any thoughts/comments?<br>
<br>
Cheers,<br>
Daniel<br>
</blockquote></div></div><div class="gmail_extra"><div class="gmail_quote"><br></div></div></div>